rta: no-op when AExpr::mtype or AExpr::cast_type is null

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

rta: new method `allocate_mtype` to factorize some code

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

Merge: A more POLA to_s on Collection

As announced in #1385, this PR change the default `to_s` on collections.

Before: `[1, 2, 3].to_s == "123"`
After: `[1, 2, 3].to_s == "[1,2,3]"`

The main issue is for code that used `to_s` on array to get a fast concatenation.
Those should now use `plain_to_s` (or `join("")`).

Close #off-line issue signaled by @ablondin

Note: this PR fails on bootstrap (`to_s` is used internally) thus requires that `c_src` is regenerated.

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

Merge: nitdoc: fix things, add more tests and benches

Two fixes:

* no more multiple instanciation of ToolContext (-25% of total nitdoc time)
* fixed JS scripts (no more bug with the search bar that does not display)

Testing:
* add more tests on pages structure
* add a script to build the documentation from origin/master version (easier to diff outputed doc after local changes)
* add a benchmark to nitdoc against the origin/master version

Performances comparison after fix on ToolContext:


(`CI-nitdoc` is busy right now on Jenkins so you will have to wait for the demo)

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

tests: update sav/nitdoc_args4.res after merge with 

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

Merge master to solve conflicts

tests: update with new `to_s` on collections

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

lib/json: update nitunits with new Collection::to_s behavior

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

lib/standard: add nitunits to Collection::plain_to_s

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

lib/standand/string: provides a better to_s for collections

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

lib/opts: clients of `errors` use `join("\n")`

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

lib/standard: format use `plain_to_s` for fast concat

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

lib/standard: StringWriter use `plain_to_s` for fast concat

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

lib_standard_array: reprovide Array::plain_to_s

Was dropped by the merge of #1400

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

Merge: friendz with Android Audio API

friendz now uses the Android Audio API
+ fixed the new_global_ref functions in Assets and Resources

Pull-Request: #1125
Reviewed-by: Jean Privat <jean@pryen.org>
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: String split

Right now, the string module is over 2500 loc long.

For the mental health of the maintainer(s), it was getting too big to understand what was going on in it.

So, this PR introduces the text group in standard, string is split in several modules inside of it:

* native: Introduces NativeString
* abstract_text: Introduction of basic types
* flat: Any array-based string
* ropes: Rope-based strings

Big con: To merge this PR, bootstrap will need to support the nullable arguments syntax.

But in the meantime, please feel free to comment on the code

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

Merge: lib/array: remove useless cast to comparable.

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

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

Merge: Kill dead nitg

Remove all remaining deprecated reference to `nitg`.

The renaming of `nitg` to `nitc` and the deprecation of `nitg` was done Wed Dec 10 so 6 month ago. cf #1000

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

Merge: Regeneration of c_src

Last one was Fri Feb 6, four months ago.

Close #1426

Pull-Request: #1455
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: ArthurDelamare <arthur.delamare@viacesi.fr>
Reviewed-by: Jean-Philippe Caissy <jpcaissy@piji.ca>

regern c_src

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

makefile: create ../bin is required

because the dir is now empty, it starts uncreated on the initial git clone

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

Merge: Kill big vararg in parser

In `parser::Parser::build_reduce_table` an array is allocated with `.with_items` that takes a vararg.

The vararg used is quite large (1116 elements) and this cause that the generated C code takes 30m to compile with clang-3.6 if -O2 is used; it is likely a regression in llvm since the same code takes appreciatively 1s with clang-3.5 (and with gcc).

Unfortunately, clang-3.6 is currently provided by Apple to MacOS X users.

The proposed solution is to allocate the array then use a sequence of `add`. The generated C code is more complex but clang-3.6 can compile it.

Close #1498 (once c_src is regenerated)

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

Merge: compiler: NativeString is a char* to please C library.

Commit 6b52ea3d5ca77dbcc6edbbf71c1aa9419a019bac  on chars and string changed the ctype of NativeString to `unsigned char*`. This was the cause of C warnings `-Wpointer-sign` because `unsigned char*` is not a subtype of the universally used `char*` that is activated by default on clang.

So the change is reverted and NativeString become again `char*` (note that Byte and Char ctypes are not reverted)

close #1499

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

Merge: example: add boolean usage example from rosetta code

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

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

man: update title of nitc to `NITC`

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

c_src: rename the primitive compiler to `c_src/nitc` everywhere

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

nittester: use new compiler engine names (`nitc*`)

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

tests: rename compiler engines with `nitc*`, not `nitg*`

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

bin: remove old transitionnal script bin/nitg

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

tests: use `nitc` in the `nitc.args`

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

misc/bash_completion: remove hack for nitg

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

lib/emscripten: talk about `nitc` (not `nitg`)

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

vim: rename `nitg` checker to `nitpick`

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

remove bin/Makefile (?!)

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

Merge: nitunit: use the Markdown parser from `lib/markdown`

Since the introduction of `nitmd`, `nitunit` was still using the old `docdown` parser.
This PR do the migration.

There will be a lot of unrecognized tests since the old paerser allowed between four and five spaces before a nitunit, the new one enforce the rule of 5 (couting the space after the `#` comment token).

Before:
~~~nit
#    assert true
~~~
Was recognized as nitunit, but if you ignore the `# ` string as prefix of each comment, there is only 3 spaces before the nitunit.

After:
~~~nit
#     assert true
~~~
We need 4 spaces before the nitunit plus the `# ` prefix, this gives us 5 spaces.

Note, this is already the behavior followed by Nitdoc.

Should fix #1331

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

Merge: nitrpg: remove points if a PR is closed without merge

So @ventilooo won't cheat again!

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

Pull-Request: #1477
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: ArthurDelamare <arthur.delamare@viacesi.fr>

Merge: rosetta_code: sha1

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

Pull-Request: #1456
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: ArthurDelamare <arthur.delamare@viacesi.fr>

Merge: Nitiwiki next

Some fixes for nitiwiki in prevision for the nitlanguage.org site migration and the resolution of #824.

Do not consider b481cea since it belongs to #1368 (and break the loader)

Feature summary:
* ini accept array notation
* customizable sidebar content
* customizable sidebar position
* wikilinks support external links
* better search by title
* better search by path
* make breadcrumbs and summaries optionnal
* add link to github for last_changes and edit mode

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

parser: regen parser.nit without the big vararg

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

parser: `build_reduce_table` uses a sequence of add instead of a `.with_items`

The big vararg generated produce C code that takes 30m to compile with
clang-3.6 because of a regression (the same code takes less than 1s with
clang-3.5).

Unfortunately, clang-3.6 is currently provided by Apple to MacOS X users.

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

misc: add build_origin_doc to easily build reference document from master

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

benchmarks: add nitdoc benches

Used to compare prefs between HEAD version and origin/master version

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

nitdoc: test pages structure

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

compiler: NativeString is a char* to please C library.

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

nitdoc: better pretty-printing

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

benchmarks: add benches for nitunit

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

lib: fix whitespaces in misc files

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

lib: fix nitunits indentation

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

lib: fix nitish in nitunits

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

src: remove unused docdown parser and test

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

nitunit: update tests

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

nitunit: use the Markdown parser from `lib/markdown` instead of `src/docdown`

Fixes #1331

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

lib/markdown: fix multiple fence in the same comment

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

example: add boolean usage example from rosetta code

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

nitrpg: remove points if a PR is closed without merge

So @ventilooo won't cheat again!

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

rosetta_code: sha1

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

Merge: Activate C warnings in FFI code and revert `NativeString` to `char *`

The revert commit is partial, it affects only NativeString and does not touch Char.

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

Merge: Work on the Curl module

Changes:
* Simplify callbacks so they do not break light FFI only engines.
* Remove about 8 classes and 3 C structures.
* Remove the custom implementation of NativeFile.
* Update style to the latest best practices. Note that it was one of the first large module based on the FFI, a lot has changed since then.
* Revamp and rename CurlMail, its API is now nicer to use:
~~~
import curl

var mail = new CurlMail("sender@example.org", ["to@example.org"], cc=["bob@example.org"])

mail.subject = "Hello From My Nit Program"
mail.body = "<h1>Here you can write HTML stuff.</h1>"

mail.headers_body["Content-Type:"] = """text/html; charset="UTF-8""""
mail.headers_body["Content-Transfer-Encoding:"] = "quoted-printable"

# Set mail server
var error = mail.set_outgoing_server("smtps://smtp.example.org:465", "user@example.org", "mypassword")
assert error == null

# Send
error = mail.execute
assert error == null
~~~

Thoughts on what's to do next:

* Extract the request API so Curl is only an implementation. The abstract API could also be implemented on Android where we can use Java services instead.
* Clean up `CurlHTTPRequest`.
* Use `Error` and differentiate between internal Curl errors and normal transfert failures.
* Replace the C structure linked to reading data and add tests for its usage.
* Add dead simple services: `Text::download: nullable String`, `Text::download_to_file(filepath: Text)`, `Text::download_to_stream(stream: Writer)` or something like that.
* Make more classes of native_curl private as we cannot make the module private because it defines the user-customizable callbacks. Some modules (neo4j) use what would be private, so it requires updating them also.

---

Should fix #936 and other problems when importing but not using Curl.
Also helps #1443 correctly report unsupported programs.

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

Merge: Add some collections for java/collections.nit

This PR complete the java collections : ``` int[], short[], long[], boolean[], byte[], char[] ```

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

Merge: Attribute and autoinit annotations

A big set of loosely coupled but related changed on the annotations on attributes and methods.
Here is a summary of the changes on the user-side:

* No more special case for the kind of classes, so you can use `autoinit` in interfaces.
* Explicit annotation `autoinit` is understood on attributes and is the default on value-less concrete attribute.
* Abstract attributes are `noautoinit` by default, if `autoinit` is given, it is applied on the setter. This way abstract attributes are basically just a couple of getter-setter. close #1311

This way there is less special cases and behaviors. Still more work is required to finish #1322.

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

Merge: nitvm: Basic blocks generation and SSA-algorithm

This PR introduces a SSA-algorithm implementation in the nitvm.

First, basic blocks are generated. A basic block is a sequence of instructions without a jump inside.
In this version, basic blocks are generated from the AST and so they contain only the first instruction and the last one.

Then, SSA-algorithm (Single-Static Assignment) is implemented. In SSA algorithm, variables are renamed to have only one assignment per variable.
Each time an assignment is made, a new version of a variable is made.

Some variables (phi-functions) have several assignment and so, several dependances.

The basic block generation and SSA are computed for each method in a lazy way in the nitvm.
The main objective of SSA is to give the dependances for each variable (this will be used in further PR).
This module also collect all object-mechanisms sites during the visit of methods.

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

Merge: example: add 24 game task of Rosetta code

Rosetta code example of 24 game
<http://rosettacode.org/wiki/24_game>

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

Merge: file: fix `simplify_path` behavior for path starting with `.`.

Before this PR `./` was simplified in `/` which was wrong.

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

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

Merge: Date Lib

les fonctions ce trouve dans le module date_f.nit.

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

lib/markdown: fix missing space after wikilinks token

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

nitiwiki: add github url for last changes and edit mode

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

nitiwiki: also search relative patch in current section

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

nitiwiki: search titles in lower case

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

nitiwiki: make breadcrumbs optional

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

nitiwiki: make summary optional

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

nitiwiki: allow absolute url in wikilinks

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

nitiwiki: introduce customizable sidebar

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

nitiwiki: make all wiki entries able to parse markdown

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

nitiwiki: fix link lookup on root

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

ini: accept array notation

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

ini: show error instead crashing on bad formatting

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

nitiwiki: change position of sidebar from config

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

nitvm: Using SSA module in the nitvm

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

nitvm: A method compile offers a unique entry point for compilation-like mechanisms

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

nitdoc: remove `require.js` dependency

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

file: fix `simplify_path` behavior for path starting with `.`.

Before this PR `./` was simplified as `/` which is wrong.

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

lib: fix C compiler warnings in sha1, socket and sdl

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

lib: fix C compiler warnings in standard library

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

Partial Revert "compiler: Changed types of Char to uint32_t and NativeString to unsigned char*"

This reverts commit 6b52ea3d5ca77dbcc6edbbf71c1aa9419a019bac.

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

src: compile user C code with -Wall

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

example/simple update sounds test

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

contrib/friendz: update so it uses android sound library

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

android assets and resources update

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

android audio : update with error management

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

tests: add base_attr_annot.nit

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

tests: add base_attr_abstract[34].nit

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

tests: update other tests related to init with setters

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

tests: add base_init_setter.nit

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

tests: update base_attr_abstract[12].nit

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

modelize: remove restriction on concrete class in intializer collect

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

modelize: error/warning if explicit autoinit on attributes

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

modelize: error in `autoinit` is used in class refinement

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

modelize: move `autoinit` management on method into AMethPropdef

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

modelize: rely on `AAttrPropdef::mreadpropdef` to be the main property

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

modelize: add `AAttrPropdef::mtype` to factorize the type access

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