vim: look for `{ in deciding to highlight ffi language name. eg "C".

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

Merge: Nitdoc: some fixes and cleaning

* Fixes the topmenu brand display
* Update bootstrap version to v3.3.4
* Avoid nesting of DefinitionArticle to facilitate tabular display (next PR)

Demos are ready: [stdlib](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/stdlib/index.html) [nitc](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/nitc/index.html)

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

Merge: Fix calls on primitive receivers

Fix #1301

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

tests: add base_primitive_recv.nit

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

compiler: use `adapt_signature` to simplify and clean `table_send`

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

Merge: Semicolumn

Propose that `;` is usable as a hard line break. While this is not really nit-ish, the only need is in fact to be able to write short one-line scripts in environment where linefeeds are not an option.

~~~sh
$ ./nit -e 'for line in stdin.each_line do; var xs = line.split(":"); if xs.not_empty then print xs.first; end' < /etc/passwd
~~~

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

Merge: Balanced brackets

The PR started as a simple implementation for the task http://rosettacode.org/wiki/Balanced_brackets but some improvements on the lib and tests.sh where also added

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

nitdoc: do not nest DefinitionArticle

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

nitdoc: update bootstrap version

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

nitdoc: fixes double span on topmenu brand

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

tests: update sav/nitunit_args6.res because `;` is no more a bad character

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

tests: add syntax_semi*.nit

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

misc_analysis: warn on superfluous `;`

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

parser: regenerate

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

grammar: distinguish `nd` for doc and `no` for optional eol

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

grammar: add `;` as an alternative hard line-break

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

Merge: nitdoc: migrate the DocPage rendering to composite rendering

Also delete old nitdoc components.

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

example/rosettacode: add task balanced_brackets

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

lib/math: add `Collection::to_shuffle` and `Array::shuffle`

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

lib/math: nitunit `Collection::rand`

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

lib/mnit: remove `MNIT_SRAND` and use `NIT_SRAND` instead

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

lib/standard: use `NIT_SRAND` envvar to control the random number generator

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

src/doc: fix anchor url following the rendering migration

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

src/doc: clean old components

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

src/doc: migrate showsource links to new templates

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

src/doc: move linearization list to `doc_phases`

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

src/doc: migrate Index page rendering to `doc_phases`

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

src/doc: migrate Overview page to `doc_phases`

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

src/doc: move rendering from old templates to new phases

Next commits will do some fixups:
* redo showlinks
* redo Overview
* redo FullIndex page

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

src/doc: migrate sidebar custom lists to new templates

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

src/doc: migrate sidebar to new templates

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

Merge: Prettier pretty printing

Some bugfixes, improvements and workaround for nitprettty.

The two options `--no-inline` and `--line-width` allow to control some rules about line-breaks.
This can be useful especially for people that want more relaxed rules.

~~~sh
$ ./nitpretty --check ../lib | wc -l
224
$ ./nitpretty --check ../lib --line-width 0 --no-inline | wc -l
188
~~~

So 36 more files are identical to their pretty printing with those relaxed rules.

For information, the tool crashed on some files in lib with the version before the PR.

Note that a lot of more work is still needed, one can see very strange and inconsistent results if the line width is forced to 1.

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

Merge: Fix a bit of everything: typos, doc, android, bucketed_game and vim

Fix typos in doc and help, improve doc, fix error in Android projects, clean up bucketed_game and add an option for #1300.

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

Merge: nitdoc: start using DocComponent rendering

This PR starts the migration of HTML rendering from old components to new DocComposite rendering.

Some minor differences can be seen in the output.

Démos: [stdli](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/stdlib/index.html) and [nitc](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/nitc/index.html).

Pull-Request: #1299
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Romain Chanoir <chanoir.romain@courrier.uqam.ca>

Merge: Filereader fix

Since 39fcf4a75aaec646f0b60d404dfcc5cb63dcb473 and because people prefer the Python/Ruby semantic to the C semantic, eof is blocking.

This PR rewrite `BufferedReader.read` to follow the Python/Ruby semantic: if there is 1 character in the system buffer, and that the programmer asks for 2, the program will wait to have 1 more character, that can block the whole program if we are waiting the missing character from some keyboard, pipe or tcp connection.

Here a comparaison of the various specifications:

The protocol is the following: I `read(4)` bytes from stdin and print them. On stdin, I write `ab\n` (3 bytes) then `cd\n` (3 other bytes) and see when the the read does Its job.
In the following, I only give the outputs since the input will be the same and adding it will require to distinguish them with the output.

~~~sh
$ ./nit_old -e 'print sys.stdin.read(4)'
a
~~~

WTF? Why only char `a` was printed? OK the implementation before the PR was more buggy than expected.

~~~sh
$ ./nit_new -e 'print sys.stdin.read(4)'
ab
c
~~~

As I expected, nothing is printed after the first `ab\n` because exactly 4 bytes are waited for, then printed. Thus we get `ab\nc`. The extra `d\n` are lost in the buffer of Nit.

~~~sh
$ ruby -e 'puts $stdin.read(4)'
ab
c
$ d
bash: d : commande introuvable
~~~

Same behavior with Ruby, so nothing is printed after the first `ab\n`.
Nice surprise however, the extra `d\n` are not lost but kept in system buffer, so still available (and read) when the shell take back the control. I am not really sure which behavior I prefer. The Ruby one might be saner from an OS point of view; but since I was surprised, one can assume that the POLA level is not that high.

~~~sh
$ python <(echo 'import sys;print(sys.stdin.read(4))')
ab
c
~~~

So exactly the same behavior than Nit with this PR.

Close #1264

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

Merge: i18n annotation

So here's the POC of a new annotation for the compiler: i18n !

Right now, you can append it to a module declaration with the locale of your choice.

It then generates a .pot template file and a .po file for the language you chose when defining your module.

The generation of the corresponding .mo (using msgfmt) file is still of the responsability of the user.

There's still several modifications to do for real usability, namely the support of superstrings, the possibility to untranslate a literal string via some kind of notranslate annotation (though if no translation is found, the key is kept as argument, so this should not be an issue).

So now, when using another value for the $LANGUAGE environment value, a different message will be printed when executing a Nit program, have fun !

NOTE: I have not yet added a test since I'm not sure how we should proceed. I guess we could just try to generate a .po file and .mo file for the example with a bash script and change the locale on the test server, I'm not sure how to do so, if you have an idea, please let me know.

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

nitpretty: use parse_full to allow groups and cie

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

pretty: force newline before a tab (or else things are ugly)

There is still issues but this is more usable anyway.

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

nitpretty: new option --no-inline

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

nitpretty: new option --line_width

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

pretty: workaround a bug with a missing `else`

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

pretty: handle typed literal arrays

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

pretty: skip commas in ANodes

I do not know why and by who the comma is skipped but in some case it
is still here. So force its removal if any.

For instance

~~~nit
foo(a, [b])
~~~

caused some issue

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

pretty: add consume_comment to process optional comments

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

pretty: add `visit_args` to factorize the processing of arguments

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

version 0.7.4

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

src/android: remove configChange section from XML, it is defined by libs

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

misc/vim: add option to disable omnifunc

Fix #1300.

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

lib/collection: better doc for Iterator

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

lib/java: fix typo

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

src/frontend: fix typo

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

lib/bucketed_game: revamp constructor of `Buckets`

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

examples: Added i18n example and po files for several languages

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

compiler: Added i18n phase and annotation

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

lib: Added gettext basic support

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

doc/src: GraphArticle use new templates

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

Merge: Named arguments

Named arguments allows the caller to specify some subset of arguments it wants to call.
This PR introduce a syntax for callers inspired from the Python one (the Ruby one uses columns that clashes with types declaration).

~~~nit
format(linewidth=80, separator=":", indent=true)
~~~

The specification tries to be as simple as possible

* Only default parameters can be passed by name; because the other are mandatory.
  This restriction is also useful to avoid to specify additional rules, like the last parameter of assignment methods, because the ones on default arguments apply.

~~~nit
fun foo(a,b: nullable Int) do ...
foo(b=1)
foo(null,1) # equivalent
~~~

* Named arguments can appear anywhere (not necessarily at the end)

~~~nit
fun foo(a: Int, b: nulable Int)
foo(b=1,2)
foo(2,1) # equivalent
~~~

* The order of evaluation is the order of the arguments (not the order of the parameters).
  I think this is more POLA this way.

* The names to use are the one of the statically designated method declaration (not the introduction).
  This one is problematic because the language permit the name of parameters to change in redefinitions.
  But in case of errors or in a IDE it is often more useful to point the most precise method definition.

The implementation was trivial because the hard work was already done in the previous PR.

This PR was unlocked by #1280

Pull-Request: #1281
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Etienne M. Gagnon <egagnon@j-meg.com>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Romain Chanoir <chanoir.romain@courrier.uqam.ca>

Merge: Parallelization phase improvement

Adds the support of methods with return value

If you use `is threaded` on a method with a return, it will instead return you a custom instance of `Thread` class on which you can use the method `join` to retrieve the return of the annotated method. `join` waits for the method to end before returning.If you want to make sure that the method has done his work before trying to retrieve it's return value, you can check it with the boolean `is_done`

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

tests: add base_arg_named_inherit.nit

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

tests: add base_arg_named.nit and base_arg_order.nit

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

engines: process named arguments

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

typing: process named arguments

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

parser: regenerate

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

grammar: add expr.namedargs

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

grammar: extract `vararg` in a specific arg production

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

model: add `MSignature::mparameter_by_name`

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

src/doc: InheritanceList use new templates

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

src/doc: IntroRedefList use new templates

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

src/doc: HierarchyList use new templates

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

src/doc: DefinitionArticle use new templates

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

src/doc: ConcernsArticle use new templates

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

src/doc: IntroArticle use new templates

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

tests: Added test for correct semantics of read in BufferedFileReader

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

lib/standard: Fixed read method of BufferedReader

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

pretty: make some method more robust (or more informative when crashing)

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

Merge: More binary operators

The first part of the PR cleanup and factorize some things related to the operators in the grammar and the AST.
Then the four bitwise operators are added.

Close #1271 unless some more thing is added to it (like generalized unary operators)

Note: as usual, github is useless when the parser is regenerated, so review the individual commits if you prefer.

Pull-Request: #1295
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

tests: clean and complete test_ffi_c_operators

thanks nitpretty!

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

tests: update error_syntax2.nit because & is now an operator

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

tests: improve (and rename) test_operators.nit

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

src: handle new bitwise operators

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

Merge: nitdoc: prepare the migration of nitdoc component to new phases

Only internal modification, no diff with the generated HTML of the previous version.

If you don't trust me (I you shouldn't), the demos will be [there](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/stdlib/index.html) and [there](http://gresil.org/jenkins/job/CI-nitdoc/ws/doc/nitc/index.html) in a few minutes.

> Brace yourself, next PR on that subject will be big.

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

Merge: nitrpg: periodize stats

This will be used to display periodic stats about players and games.

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

Merge: Nitrpg work

Improve the dashboard on the player page with `request_for_comment` things, and assigned work

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

Merge: Hint vararg

Improve the error message so that user that try to pass an array as is instead of elements is hinted about the `...` reverse-vararg operator.

~~~
fun foo(xs: Int...) do end
foo(1,2,3) # OK
var a = [1,2,3]
foo(a) # Error, expected `Int`; got `Array[Int]`. Is `...` missing?
foo(a...) # OK
~~~


close #98 (quite old bug)

Pull-Request: #1294
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: Some metrics improvement and cleaning

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

Merge: Model: add mclassdef2node

This was missing, I don't understand why but I need it.

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

tests: update sav/base_vararg3* with new hint error message

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

typing: special message when a single vararg is refused bu could be `...`-ed

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

contrib/nitrpg: use stats to store player nitcoins

So podium and stats can be displayed by time periods.

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

contrib/nitrpg: make stats periodized

Currently stats are stored:
* all time
* yearly
* monthly
* daily
* weekly

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

lib/counter: introduce Counter::dec

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

src/doc: introduce DocComposite HTML rendering services

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

metrics/mendel: add missing comments

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

metrics/mendel: use new constructors

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

metrics/mendel: sort results in console display

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

metrics: fixe missing documentation warnings

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

metrics: remove useless return types in redefs

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

metrics: move `sum` up to interface `Metric`

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

metrics: allow Metric sorting

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

lib/standard: introduce ReserveDefaultComparator

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