misc/vim: always use the standard library for autocompletion

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

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>

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>

nitdoc: introduce StructurePhase

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

nitdoc: introduce ConcernsPhase

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

nitdoc: introduce MakePagePhase

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

nitdoc: introduce ExtractionPhase

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

nitdoc: migrate nitdoc from `doc_pages` to `doc_phases`

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

nitdoc: create `doc_phases` group

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

nitdoc: introduce `doc_base` module

This module introduces new Nitdoc concepts:
* DocModel
* DocPage and DocComposite
* DocPhase

These concepts will be used in further commits to "phasify" Nitdoc.

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

contrib/nitrpg: gitignore binaries, cache and game data

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

contrib/nitrpg: add a Makefile

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

contrib/nitrpg: introduce the tool that run the website

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

contrib/nitrpg: introduce templates to render the game as an HTML website

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

contrib/nitrpg: add a listener tool to update the game from GithubEvents

This listener must be running to keep the game up to date.

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

contrib/nitrpg: add some basic reactors

Here the reactors are used to collect new pull request and comments to reward the player with nitcoins.

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

contrib/nitrpg: enable statistics collection from GithubEvent

This shows how to implement GameReactors.
Statistics will also be used latter in the web display.

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

contrib: introduce nitrpg base structures

Nitrpg is a rpg game coded in nit that works with Github.

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

compiler: use global constant instead of literal to indicate file-name in errors

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

loader: attach mmodule to sourcefile

So the associated module can be retrieved from any node

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

Merge: GitHub misc

Some fixes for the Github API.

Also introduce `Repo::search_issues` which allow to list specific issues with an advanced query.

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

Merge: Optimize get_or_null

Recent profiling of the interpreter nit shows that a lof of time is used in hashmaps. This PR improves the standard use-path with the safe method `get_or_null` that is more efficient (and simple) than `has_key` followed by `[]`.

The time for `nit doors_with_classes.nit 5000` give

before: 0m5.592s
after: 0m5.208s (-6,7%)

Note: by comparison, the ruby version is 0m2.284s, and the python is 0m0.116s ; there is still a lot of work

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

Merge: jwrapper: interfaces, "Compiled from" line and output file without .nit

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

Merge: Better Vim autocomplete by scanning all imported modules

The new Vim plugin uses `nitls` to locate all Nit modules relevant to the current directory. This is a nice way to support both complex projects and new incomplete modules.

The recommended usage is to update the list of modules when launching Vim and then do it manually afterwards. You may want to update it after adding an importation and saving the file. It could also be updated on each file save or loading but it gets pretty slow. Actually, the whole system may be too slow for older computers (sorry @R4PaSs) or those running inside a slow VM (sorry @BlackMinou and @MehdiAit).

A nice feature to have, would be to call `nitls` on the standard library when there's not local Nit source file or project. It occurs when creating the first Nit source file in an empty directory. @privat, what would be the best way to do that? Could we have `nitls -M standard`?

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

loader: get_mgroup look in path

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

model: precise contracts for `lookup_*_definition?`

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

github/api: add `search_issue` service in Repo

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

github/api: move `html_url` to GithubEntity since it's shared by all of them

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

github/api: return an empty list instead of crashing for issues without labels

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

github/api: uniformize `load_issue_event` implementation with other services.

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

rosetacode: improve doors_with_class (add optional argument)

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

lib/more_collections: simplify by using `Map::get_or_null`

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

model: use `Map::get_or_null` for `get_x_by_name`

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

model: move guards `lookup_first_definition` after cache miss

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

lib/standard: redef a fast path for `HashMap#get_or_null`

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

jwrapper: accept interfaces

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

jwrapper: accept output files that do not look like Nit modules

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

misc: rename README to README.md

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

misc: intro a vim plugin with better autocomplete

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

jwrapper: deal with the optional "Compiled from" line in the grammar

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

Merge: Fix virtual types in string

`SELFTYPE` in String is used to dintinguish buffers from strings. Unfortunately it is sometime badly used.

This series try to at least improve thing so that operations on buffers do not crash at runtime.

Still, I think there is something fishy in the String vs Buffer API

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

Merge: Set accept null

Since keys of Maps can now be null, it enables the same rule with the Sets

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

Merge: parser: slight improvement of the README

Copy a sentence from a page of the website before removing it.

nitdoc will soon nicely presents components of the compiler ; so just move existing doc in a single place make sense.

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

Merge: Clean up to prepare for NitActivity

This PR cleans the way to replace our current use of `NativeActivity` by our own `NitActivity` a direct subclass to `Activity`. The result of this PR that we are less dependent on `NativeActivity` and the `native_app_glue` module.

Notable modifications:

* Use Java `Activity` instead of `NativeActivity` wherever possible.
* Rename, to follow convention, Nit `NativeActivity` to `NativeNativeActivity` and `Activity` to `NativeActivity`. Thus all previous Java code using the Java `NativeActivity`, will now use Java `Activity`. The name `NativeNativeActivity` is weird, but temporary as the class will be removed once we have our `NitActivity`.

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

parser: slight improvement of the README

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

version 0.7.1

lib/android: Bundle do not need a context or to import native_app_glue

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

lib/android: extract activities from native_app_glue

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

lib/android: in Java, use Activity instead of NativeActivity

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

Since keys of Maps can now be null, it enables the same rule with the Sets

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

string: some methods sould return String instead of SELFTYPE

because it is what they returns

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

string: fix SELFTYPE in String and Buffer

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

string: do not redef SELFTYPE in FlatBuffer

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

update NOTICE

Merge: Ropefix

A few modifications to ropes, I discovered that the `+` routine used to create too many Concat nodes passed a certain point, which inceased the RAM use of Strings unnecessarily (might help with #1106's example program, on my machine the memory went from 200M+ used to 20M).

While I was at it, I removed a few old-style-inits that were no longer necessary.

Some others however I could not remove, maybe there is a way I can do so, but with the spec of the new constructors I'm not entirely sure how to do so without retaining unnneeded information or complexifying instanciation, that'll have to wait.

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

Merge: ldflags for Android

For the same module, cflags and ldflags can vary per platform. This new annotation on annotations targets a platform for each cflags or ldflags. By default, the target platform is that of the host. It will probably need to be clarified on more modules once we support more platforms.

It will be used to load (or not load) OpenGL ES v1.0 vs 2.0 and libandroid in future PR. For now, only `calculator` should save on linking with OpenGL.

I hesitated between different annotations before settling on `@android`:
* `@target(android)` makes it clear that it is the target, but it gets a bit heavy to read: `module android_opengles1 is ldflags("-lEGL -lGLESv1_CM")@target(android)`
* `@only(android)` makes it clear that only Android is affected by this annotation.
* but `@android` is easy to read and clear enough to fit with an already complex syntax.

TODO `cflags@android` declarations are accepted but unused at this time.

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

Merge: Optimizations in nitvm

This PR introduces some change in the implementation of object mechanisms in the nitvm.

Now, the dispatch between several mechanisms is made directly in the AST nodes instead of in runtime mechanisms themselves.
This PR is preliminary work before the implementation of complex optimization systems, but now the nitvm is sligtly more efficent than before because the choice between implementations is only made once (the first time).

I use a workaround in AST nodes to avoid conflict with refinements between the interpreter and the vm.
In the future this will need to be well implemented, maybe by creating a sort of ```CallSite``` for subtyping test and attribute access and store informations into them instead of into the nodes.

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

lib/standard/ropes: Removed a few old-style-inits

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

lib/standard/ropes: Fixed concatenation routine for Concat + FlatString

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

nitvm: Optimize method call by dispatching between implementions in CallSites

Signed-off-by: Julien Pagès <julien.projet@gmail.com>

lib: update portable modules with ldflags specific to Android

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

lib: define Android ldflags in user modules

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

nitc & lib: intro the @android annotation and use it in nitc

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

nitc: refactor MModule cflags and ldflags to support different target platforms

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

src: each platform has a name, except for the default

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

nitvm: Optimize attribute access and subtyping test by orienting between implementations in AST nodes

Signed-off-by: Julien Pagès <julien.projet@gmail.com>

niti: Allow to redefine AST nodes in other engines, add documentation in vm

Signed-off-by: Julien Pagès <julien.projet@gmail.com>

Merge: A collection of miscellaneous commits

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

Merge: Android support improvements

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

examples/calculator: revamp style of the calculator code

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

C FFI: fix line offset in gcc errors on custom header code

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

lib/egl: add `renderable_type` related features

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

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

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

lib/gtk: copy string returned from `GTKEntry`

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

lib/sdl: intro `has_mouse|input_focus`

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

lib/mnit_input: add `KeyEvent::name`

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