src/doc: move remaining services from `doc_model` to `doc_base` and delete the module

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

src/doc: move HTML output related services from `doc_model` to `html_model`

So HTML related things are only imported by the HTML rendering phase.

Next commit will finish the `doc_model` cleaning by merging remaining services into `doc_base`.

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

src/doc: use `nitdoc_name` rather than `nitdoc_id` for debug name.

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

src/doc: introduce `html_templates` group.

This group will be used to store all the HTML output related things.

Next commit will extract HTML related services from module `doc_model`.

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

src/doc: make ImportationListSection and InheritanceListSection two MEntity articles since the lists are about a specific MEntity

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

src/doc: make GraphArticle a MEntity article since the graph is about a specific MEntity

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

src/doc: rename `DocComposite::add` into `add_child` to avoid conflict with `Template::add`

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

lib/android: add the gamepad module

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

lib/android: move all android annotations to the `aware` submodule

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

lib/more_collections: add `clean` to `HashMap[23]`

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

lib/json: converting JsonValue to float or int casts between numerics

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

Merge: Nitrpg: introduce game events

Events allow us to keep track of the history.

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

Merge: Nitrpg: minor fixes

Some fixes before next feature.

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

Merge: Nitrpg: minor fixes

Some fixes before next feature.

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

Merge: Rename Streams

Since #466 is not closed yet, here's the follow-up to PR #932.

This one focuses only on introducing new names for Streams, IOS and its derivatives were judged as confusing, so here's a proposition of names for the different Streams of the standard library.

Please let me know if some names are not very explicit, I'll change them if necessary.

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

Merge: lib/glesv2: intro framebuffer related services and try a new style

The main point of this PR is to try a new naming convention for the glesv2 module.

Ideas behind this convention:
1. Be as close as possible to the C API: use the same function and macro names.
2. Modify those names only for compatibility with the Nit language: the macros cannot begin with `GL_` so use `gl_` instead.
3. Apply types and other Nity features when they can help and don't interfere with the original API.

It differs from the previous convention of renaming all entities to the Nit style. It is not heavier in the module definition but should be easier for the users.

In practice, to convert OpenGL ES C code to Nit you would need to change the prefix `GL_` to `gl_`, and remove `;`.

In C:
~~~
glBindFramebuffer(GL_FRAMEBUFFER, my_fbo);
glBindRenderbuffer(GL_RENDERBUFFER, my_renderbuffer);
return glCheckFramebufferStatus(GL_FRAMEBUFFER) == GL_FRAMEBUFFER_COMPLETE;
~~~

In Nit:
~~~
glBindFramebuffer(gl_FRAMEBUFFER, my_fbo)
glBindRenderbuffer(gl_RENDERBUFFER, my_renderbuffer)
return glCheckFramebufferStatus(gl_FRAMEBUFFER) == gl_FRAMEBUFFER_COMPLETE
~~~

What do you think of the new naming convention? I would convert the whole module to it.

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

contrib/nitpg: make player list more easy to read

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

contrib/nitrpg: introduce `link` service in some GameEntities

This should lighten html strings

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

contrib/nitrpg: display events in home and player page

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

contrib/nitrpg: move logging to `listener`

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

contrib/nitrpg: PlayerReactor generate events

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

contrib/nitrg: fix link to puller in `PlayerReviewsPanel`

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

contrib/nitrpg: introduce `GameEvent`

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

contrib/nitrpg: introduce Game::save_in

Also changes the Game::save documentation

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

contrib/nitrpg: also reward player for merged commits.

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

contrib/nitrpg: introduce `link` service in some GameEntities

This should lighten html strings

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

lib/standard: Updated documentation of Streams to fit new names.

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

lib/standard/stream: Renamed streams for more explicit denomination

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

model: add MProperty::is_autoinit to identify method used as initializer

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

Merge: Vim omnifunc diffentiate between types, class declaration and instanciation

The main improvement is the knowledge of constructors. They will be suggested along with their class after a `new`, ex: `new Arr`, it will autocomplete to `Array[E]`, `Array[E].with_items`, etc. It will also show their corresponding documentation.

Also detect the keywords `isa` and `as`.

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

Merge: Link boost

Follow up of #1093

This time some vft optimizations with the option `--substitute-monomorph` that replaces monomorphic trampolines with direct call.
Late-binding is implemented with *trampolines*, that are small functions that just select and jump the to right implementations.
If, at link-time, is it known that the target will always by the same implementation then all calls to the trampoline are replaced by direct calls to this single implementation.

Note that using trampolines as indirection slows down the executable.
However, it is expected that the gain of monomorphic direct-calls overcompensates the additional indirections in polymorphic trampoline-calls.

This PR first implements trampoline then build the substitute thing on it.

For times (nitc/nitc/nitc)

* default: 0m7.216s
* --link-boost: 0m7.024s (-2,7%)

Not that impressive in fact :( but anyway this PR is still required for the next round of link-time optimizations

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

fixup tramp

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

bench_engines: add `bench_linkboost` to bench --link-boost optimizations

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

sep_compiler: add alias option --link-boost

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

sep_compiler: table_send take the original entity instead of the color

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

sep_compiler: add option --substitute-monomorph to substitute monomorph trampoline calls at link-time

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

sep_compiler: optionally use a trampoline before the vft call

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

sep_compiler: SeparateRuntimeFunction save its arguments

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

lib/glesv2: add glCheckFramebufferStatus

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

lib/glesv2: add glHint

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

lib/glesv2: add glBindFramebuffer and glBindRenderbuffer

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

misc/vim: update README with latest features

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

misc/vim: use shorter … char

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

misc/vim: use separately class types, constructors and class declarations

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

nitpick autocomplete: differenciate between class types, constructors and class declarations

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

nitpick autocomplete: do not list properties beginning with @ or _

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

Merge: nitrpg: more statistics

Some changes on the stats API and new stats for Players.

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

sep_compiler: add RuntimeFunction::c_fun_ptr

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

Merge: Detect covariance

After a slow cleanup time, publication of the tool to detect and categorize the static usage of covariance in Nit programs.

@jpages yeah!

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

Merge: Regen c_src without libunwind

The last `c_src` is 3 month old, it seems to be usual to have 3 to 4 months between updates.

This update also did not depends on libunwind: so no stacktrace but can compile on rasperry pi. so fix #1149
Note that the following steps of the initial make will still implicitly include libunwind, so on raspberry, either do the next steps manually with `--stacktrace none` or edit `src/Makefile` to add `--stacktrace none` to both `NITCOPT` and `OLDNITCOPT` (at the begin of the file)

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

Merge: Vim omnifunc for Nit

Alternative autocompletion for Nit. This one is triggered by a call to
the omnitfunc, usually `ctrl-x ctrl-o`. It will show context
appropriated modules, classes and properties. It shows all properties,
but it does not use static types to show the methods available on the
specific receiver. It also displays some doc!

It uses `nitpick` to generate metadata files analyzed by the Vim plugin.
So in combination with `syntastic`, each time you save, and the program
is valid, it updates the metadata files.

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

tests: update sav/test_test_phase_args1.res

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

tests: update sav/nitmetrics_args*.res

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

metrics: rename title of detect_variance_constraints

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

metrics: add --detect-covariance

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

contrib/nitrpg: move logging to `listener`

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

contrib/nitrpg: remove magic numbers from reactors.

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

contrib/nitrpg: clean stats display in panels

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

contrib/nitrg: fix link to puller in `PlayerReviewsPanel`

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

contrib/nitrpg: also reward player for merged commits.

contrib/nitrpg: do stats on players

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

contrib/nitrpg: do not save Game for all received events

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

contrib/nitrpg: do stats on merged commits

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

contrib/nitrpg: move statistics to GameEntity

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

contrib/nitrpg: extends GameStats API with Counter

Also makes it redef `game` like other GameEntity.

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

misc: update README on info for the Nit omnifunc

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

misc/vim: use the metadata files for a better autocompletion

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

nitpick: generate metadata files for the Vim plugin

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

test_test_phase: count projects, classdefs and methods

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

modelbuilder: add `check_subtype` and `check_sametype`

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

misc/vim: always use the standard library for autocompletion

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

c_src: regenerate

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

mkcsrc: temporally disable stacktrace (libunwind) for c_src

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

Merge: Generalize instance creation service so FFI can use it

Move up instance creation from ANew to AbstractCompilerVisitor so they can be used from the FFI implementation. Allows to call extern constructors of extern classes from extern code, and fix #1145.

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

Merge: GitHub merge: some improvements

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

Merge: Makefile: initial make use --semi-global

The produced tools in bin/ should be faster to use then.

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

Merge: Global for filenames

This PR remove the remaining big variability for nitc when generating separate files: the path of nit module in runtime error messages.
When compiling a same module but from different current working directories, the relative path of the module changed. Thus causing unnecessary difference in the generated C source code that made ccache unhappy.

Before
~~~
$ ccache -C
$ time nitc src/nitc.nit
user 2m8.264s
$ cd src
$ time nitc nitc.nit
user 2m8.676s
~~~

After
~~~
$ ccache -C
$ time nitc src/nitc.nit
user 2m9.180s
$ cd src
$ time nitc nitc.nit
user 0m9.812s
~~~

Note: some files are still different after the PR

* main.c because it contains the real filepath
* c_function_hash.c same
* _ffi.? because the pragma lines indicate the original relative file

Note 2: `.nit_compile` directories are still created by default in the current directory and let unclean. cf #792

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

Merge: String memory fixes

Another step towards the closure of #1106 and #69, this PR removes several calls to `malloc` in `lib/standard` C code and replaces them by some of the solutions proposed in #1106 to handle the allocation of Strings.

@xymus: Your review is more than welcome since we're dealing with extern code, it might be a good time to specify and document how to properly allocate extern data structures to avoid leaking code such as the ones we had.

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

Merge: nitdoc: introduces phases

This serie of commit introduces the concept of DocPhase.

The first commit introduces new concepts that are used by the other commits to move code.

For each phase, related code is extracted from `doc_pages` to the corresponding `doc_phases`.
No code is actually deleted from `doc_pages` between steps. The complete module is deleted at the end.

Refactoring is kept at a minimal level in this PR (just enough to get compatibility between old `doc_model` and new `doc_phases`).
Some hacks are used to preserve a minimum of differences in the generated HTML.

Further commits will continue the cleaning / refactoring but also change the outputed HTML, so I prefer wait for a new PR.
Demo (wait for Jenkins): http://gresil.org/jenkins/job/CI-nitdoc/

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

tests: test calling extern constructors from extern code

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

nitc: fix calling extern constructors from extern code in separate compiler

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

niti: Fixed method NativeString::copy_to in intern_call

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

lib/standard/time: Fixed String-related memory leak

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

lib/standard/string: Removed Float.to_precision_native

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

lib/standard/string: Fixed leaking Int.to_s method

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

modelize: move resolve_mtype to modelbuilder_base.nit

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

github_merge: show cover letter after successful merge.

If the merge fails, the cover letter will be shown during the commit,
so no need to show it before.

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

github_merge: do not crash if the mergeable status is unknown

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

github_merge: always to a merge commit (--no-ff)

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

Merge: loader: get_mgroup look in path

Was asked by @xymus in #1134 to be able to do `nitls -M standard`

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

Merge: contrib: Introduce nitrpg

We have been talking about this a long ago. I made some demos, merged the full Github API stack...

Tonight is the nigth!

In this first PR we introduce some game basics and just enough to show how the game is supposed to work.
You can run it as this but the game only rewards new pull requests and +1 comments.
Further PR will introduce more capabilities like Github history reload, achievements, experience...

The game is separated into two tools:

* `listener` is the Github hook that capture all the github events on a repo.
* `web` is a Nitcorn client that display the game as a HTML website.

Four first commits are from #1141, please disregard.

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

Makefile: initial make use --semi-global

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

nitdoc: remove unused module `doc_pages`

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

nitdoc: adapt `doc_model` to avoid diff with new phases

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

nitdoc: introduce IndexingPhase

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

nitdoc: introduce RenderHTMLPhase

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

nitdoc: introduce IntroRedefListPhase

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

nitdoc: introduce InheritanceListsPhase

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

nitdoc: introduce GraphsPhase

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

nitdoc: introduce POSetPhase

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