nitc/android: "handle" all config changes

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc/android: differentiate between debug and release versions

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc/android: support using -m with Android compilation

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: improve changelog.sh

Don't count merges and display number of commits

Pull-Request: #1117
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: lib/github: implements caching to maximize rate limit.

Using JsonStore to save data locally and lighten the stress on API rate limit.

Pull-Request: #1110
Reviewed-by: Jean Privat <jean@pryen.org>

lib/github: implements caching to maximize rate limit.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: Audio management for android + audio Abstraction API

Fixes some problems with the audio management for the android platform. The audio API for android still needs some work but is fine to use now !
The Abstraction API for sound management is very simple for now

Pull-Request: #1101
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

assets_and_ressources : global ref fixes

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

mnit simple : improved audio tests

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

mnit simple : change the format of dimensions to fix values

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

mnit simple : apparently, you can't name a sound file "sound"

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

App: Audio abstraction in App

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Android audio : better audio handling

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Merge: Check missing return in attributes with a block

Fixes #1103

Pull-Request: #1114
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>

changelog.sh: also count commits

Signed-off-by: Jean Privat <jean@pryen.org>

changelog.sh: shortlog with --no-merges

Signed-off-by: Jean Privat <jean@pryen.org>

Merge: Minor graphics related fixes and features

Pull-Request: #1113
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: Fix nitiwiki: Fixes auto-index links.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Pull-Request: #1112
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: lib/github: introduces Github hook events

First step to GitHubAPI hooks.

Pull-Request: #1111
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Romain Chanoir <chanoir.romain@courrier.uqam.ca>

tests: improve base_attr_init_val_block.nit to test bad returns

Signed-off-by: Jean Privat <jean@pryen.org>

typing: check missing return in AAttrPropdef with block

Signed-off-by: Jean Privat <jean@pryen.org>

nitiwiki: makes auto-index appear in test files

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitiwiki: fixes links to auto-index pages.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: introduces Github hook events

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: nitdoc: does not apply header style to span elements anymore.

Fixes #1108

Pull-Request: #1109
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: Runtime function

Refactor SeparateRuntimeFunction and related services in the separate compiler.

This simplify the compiler and improve the efficiency of the generated code a little.

Before: 0m7.588s
After: 0m7.364s (-3%)

This refactorisation is required for some next PR that will improve polymorphic calls

Pull-Request: #1102
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

nitdoc: does not apply header style to span elements anymore.

Fixes #1108

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

compiler: --erasure use `virtual_runtime_function` instead of hard-building names

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: SeparateCompilerVisitor::table_send uses services of virtual_runtime_function

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: add SeparateRuntimeFunction::c_sig and SeparateRuntimeFunction::c_ret

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: do not generate specific virtual method if compatible with the original

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: factorize VirtualRuntimeFunction and SeparateRuntimeFunction

Signed-off-by: Jean Privat <jean@pryen.org>

Merge: Better Objective-C FFI and cleanup for OS X

This PR opens the way for the iOS platform support.

Two commits may look very similar: "ObjC FFI: import public Objective-C headers for imported modules" and "nitc: fix duplication on imports of public foreign code". They both manage importing headers from other modules so public extern classes may work across module, but one is for Objective-C and the other for C. This service will have to be implemented for other languages too (mostly C++), I'll be looking for a higher level service at that time.

Pull-Request: #1105
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: GitHub api: Issue events, contributor stats, files

This time it's all about:

* issue events
* contributor stats
* files

Pull-Request: #1104
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Jean Privat <jean@pryen.org>

lib/cocoa/examples: we do not need to import framework headers anymore

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

ObjC FFI: import public Objective-C headers for imported modules

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

ObjC FFI: use bridged cast to store Objective-C objects in pointers

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc: fix duplication on imports of public foreign code

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/glesv2/examples: import sdl only from mnit_linux

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc: move platforms modules to src/platform

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/glesv2: fix ordering and compatibility in uses of glGetActive

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/sdl: fix right and middle mouse button id

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/sdl: add access to texture pixels and alpha mask

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/sdl: intro `grab_input`, `ignore_mouse_motion_events` and fix `show_cursor=`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/c: fix type of index in `NativeCArray::[]`

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

Merge: Do not display test suite modules in Nitdoc

Do not display test suite modules in Nitdoc

Fixes #779

Pull-Request: #1097
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: Namespace for

Better implementation for `full_name` and `c_name` that make them shorter and use the project as the namespace of public entities.

Examples of fullnames of entities used in messages

* A public class `A` defined in the module `m` of the project `p` was `p::m::A`, now it is `p::A`.
* A public method `x` defined in a class `A` in the module `m` of the project `p` was `p::m::A::x`, now it is `p::A::x`

Exeaple of fullnames for entities that are used internally:

* The refinement of `A` in a module `p::n` was `p::n#p::m::A`, now it is `p::n#A`
* The redefinition of `x` in a class `B` in `p::n` was `p::n#p::m::B#p::m::A::x`, now it is `p::n#B#A::x`

The `c_name` get some comparable simplifications, so C identifiers will be simpler and will less overflow in unwind or in valgrind.

Note: please ignore the first commits that come from #1069

Pull-Request: #1092
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

tests: update sav/nitunit since fullnames changed in generated files

Signed-off-by: Jean Privat <jean@pryen.org>

model: implement various c_name and  full_name with *namespace_for

Signed-off-by: Jean Privat <jean@pryen.org>

model: implements MProject::full_name and MProject::c_name

Signed-off-by: Jean Privat <jean@pryen.org>

Merge: GitHub API: handles comments

This time for comments:

* commit comments
* issue comments
* diff comments

Pull-Request: #1098
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: Websockets

Fixed websockets with the new and improved sockets.

While I'm at it, changed a bit the way it works, the old one was a bit (a lot ?) quick and dirty, so here's that.

Pull-Request: #1096
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: Nitpretty: make the tool more customizable

Added some options to control behavior.

Interesting features:

* option --break-strings to enable literal string breaks
* option --inline-do to enable do inlining
* option --skip-empty to enable empty line skipping

Pull-Request: #1088
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: Allow keys to be nullable in `Map` and `MapIterator`

I need at least the first commit since MapIterators can be used with non-maps. The second commit is more for consistency. Note that HashMap keys are still non-null.

If this is of interest to you, I'll complete and publish my `NullableMap` in `more_collection`.

Pull-Request: #1041
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: FFI use `MModule::cname`

This prevents conflicts between two modules using the FFI and with the same short name. It follows the same change for all generated files.

Pull-Request: #1039
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

model: add MModule::namespace_for and MModule::c_namespace_for

Signed-off-by: Jean Privat <jean@pryen.org>

lib/more_collections: update more maps to accept nullable keys

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib/map: add factory to Map returning a HashMap

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

lib: update HashMap and ArrayMap to support nullable keys

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc&lib: MapIterator keys can be nullable

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc: FFI and nitni use MModule::c_name

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitpretty: does not skip empty lines anymore

But we let the user chose with --skip-empty

Fixes #1057

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: does not force `do` inlining on APropdefs anymore

But also let the user chose with --inline-do

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: does not force break on string literals anymore

But still be done using --break-strings

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: displays annotation visibility

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: does not assert for location on catch_up

This simple fix avoid assert failed on catch_up.
Does a no-op if current_location is after the target location.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: fixes length calculation for statements

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: fixes display of multiple annotations

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: does not double tab on multiline statement anymore

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

nitpretty: factorizes block visit on APropdefs

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles github files

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles diff comments

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles github contributor stats

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles github issue events

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles issue comments

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: handles commit comments

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: introduces comment abstraction

Github manage different kinds of comments (in commits, issues and diff).
The abstraction is used to factorize services from these kinds.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: factorize load_from_github behavior.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/github: uniformize assertions in examples.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

lib/standard/string: Fixed substring, could produce strings with a negative length

Signed-off-by: Lucas Bajolet <r4pass@hotmail.com>

niti: Fixed primitive type NativeFile for interpreter

Signed-off-by: Lucas Bajolet <r4pass@hotmail.com>

debugger: Updated to support new Websockets

Signed-off-by: Lucas Bajolet <r4pass@hotmail.com>

lib/websocket: Refactored websocket to be more compliant with the model of the Sockets

Signed-off-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: Use linker symbols to encode colors

Genuine constant static variables are used to store the colors used in OO mechanisms.
This makes the compiler program slower since additional indirections are required to get the values.
It also produces a little bit larger executables since static memory has to store the colors.

This PR introduce a trick user in the original PRM that uses linker symbols to encode the colors.
It is not completely portable so it must be activated with the option `--colors-are-symbols`

For numbers (with the traditional nitc/nitc/nitc)
before: 0m7.544s
after: 0m7.228s (so -4%)

Pull-Request: #1093
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: GitHub api: Issues and PRs

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

Pull-Request: #1095
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: Android release mode sign and align the generated APK file

Pull-Request: #1094
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: More on Sockets and FinalizableOnce

Additionnal notes:
* `FinalizableOnce` will probably be the most common `Finalizable`, but it wouldn't cover all cases.
* Remove some unused features from `socket` to make it compatible with Android.
* Fix an illegal extern class by making it a pointer.
* Do not `exit 1` on socket error.

Pull-Request: #1066
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: neo4j: Introduce an API for graphs

You have waited for it. Now it comes: the Neo4j graph API.

## Features

* Customizable identification scheme: You have power on how IDs are generated and which property is used to store it. For the moment, only one scheme is provided: sequential IDs.
* Customizable storage mechanism: You have power on where and how your graph is saved. You can then reload your graph at will. For the moment, two storage medium are provided: Neo4j and JSON.
* Partial loading and saving: For when you do not want put all your eggs in one basket.
* Progression tracking: You can track the progression of a long-running operation (like saving thousands of nodes in a Neo4j database) and display it to your users the way you want.

## What is included

### The API itself
* progression: Add an API for progression tracking.
* neo4j: Add an API for graphs.
* neo4j/graph: Add node removal methods.
* neo4j/graph: Add optimization methods.

### Neo4j as a storage mechanism
* neo4j/graph: Add Neo4j as a storage mechanism.

### JSON storage mechanism
* json: Factorize the trivial implementation of `to_json`.
* stream: Optimize `StringIStream.read_all`
* neo4j/graph: Add a JSON storage mechanism.
* neo4j/graph: Enlarge in advance when loading from JSON.

### Sequential identification scheme
* neo4j/graph: Add a sequential identification scheme.
* neo4j/graph: Optimize some services of `SequentialNodeCollection`.
* neo4j/graph: Implement optimization services of `SequentialNodeCollection`.

### Garbage collection :smile:
* neo4j: Remove `json_store`.

## Comming soon in a future PR near you
* Refactoring of `/src/neo.nit` and of `neo_doxygen` to use the new API.

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #1062
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Merge: Fix more warnings

And some more....

Pull-Request: #1008
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

man: document `--colors-are-symbols` in nitc

Signed-off-by: Jean Privat <jean@pryen.org>

fixod

nitdoc: do not generate page for test suites.

Fixes #779

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

model: add MModule::is_test_suite attribute.

Used to discriminate nitunit test modules.

Signed-off-by: Alexandre Terrasa <alexandre@moz-code.org>

compiler: add `Platform::supports_linker_script`

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: add early `Compiler::target_platform` with a non-nullable type

Signed-off-by: Jean Privat <jean@pryen.org>

compiler: use linker symbols to encode colors

Signed-off-by: Jean Privat <jean@pryen.org>

lib/android: add a README file to the project

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc/android: sign and align the APK file in release mode

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc/android: set the icon for the APK as expected by the Play Store

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

nitc: `exec_and_check` print the command on verbose level > 1

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>

websocket: Fix for use with new version of Streams

Signed-off-by: Lucas Bajolet <r4pass@hotmail.com>