src: Added complete FlatString generation from compiler

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

lib/standard: More uses of new FlatString.full when appropriate

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

Merge: Relocate examples

Some short programs in `examples` are related to specific libraries. Therefore to improve coupling and cohesion, these examples are moved into a subdirectory of their associated libs.

Basically, it means moving `examples/foo` into `lib/foo/examples`

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

gettext: add langannot.pot to the .gitignore

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

tests: skip lib/android/examples/src/ui_test.nit on osx and nint

because now all examples of lib are tested.

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

Merge: Small fix to iOS support

Add `ONLY_ACTIVE_ARCH=NO` to the xcode call because it is required by the latest xcode.

For the iOS example, you would expect `AppKit.h` to be included by the imported module. It is, but at the same time it is ignored for performance reason since none of the imported module services are used. This will be fixed in a future PR. For now, we add the include locally as it is a better example and happen to avoid the problem.

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

tests: update listfull.sh for new examples location

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

lib: move `examples/mpi` to `lib/mpi/examples`

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

lib: move mpi to its own project directory

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

lib: move `examples/lang*` to `lib/gettext/examples`

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

lib: move gettext to its own project directory

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

lib: move `examples/pnacl` to `lib/pnacl/examples`

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

lib: move pnacl to its own project directory

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

pnacl: fix whitespaces

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

Merge: Rename `standard` as `core`

Let's see what will break

Close #1434

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

Merge: Work towards an Android testing framework and fix a few bugs

Update check_android.sh to accept any APK file, many files at once, deduce the package name from the APK and accept some command line arguments. Users can manage their own emulator (or device) and they can show the emulator window.

Rename mnit_simple to mnit_test with 2 alternatives on Android: minimal and complete. Also fix what was broken in the complete test and the lib. The complete test still doesn't work because of a bug in `android::audio` fixed by #1587 which is not yet merged.

Consider this PR as the first part for integrating Android test in jenkins. It will be easier to tweak the integration with already the script in master.

This PR does not fix the issue with some Android apps where they crash after about 30 seconds, but it could help prevent future regressions.

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

nitc: add ONLY_ACTIVE_ARCH to the xcodebuild invokation

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

lib/cocoa: cocoa_message_box import AppKit.h as it doesn't use app_kit

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

lib: move `examples/emscripten` to `lib/emscripten/examples`

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

lib: move lib/emscripten to its own project directory

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

emscripten: fix whitespaces

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

tests: update thing to `core` (instead of `standard`)

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

nitunit: special case in on `core` now

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

nitmetrics: kill unused functions `is_standard`

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

loader: load `core` by default

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

code: update references to `standard`

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

lib: add placebo `standard` to enable the bootstrap.

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

lib: rename `standard` as `core`

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

lib/standard: fix whitespaces before renaming

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

tests: remove orphaned savs

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

lib/android: fix `Bundle::string` behavior on null strings from Java

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

lib/android: use a global reference for `Bundle::native_bundle`

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

lib/android: apply audio hook on portable methods

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

contrib/mnit_test: add icons

Using the PNG version to simplify the compilation process which is much
more complex from an SVG file. And I misplaced the source...

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

contrib/mnit_test: improve documentation of `test_audio`

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

contrib/mnit_test: deactivate the test reading `long` from shared_preferences

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

contrib/mnit_test: deactivate the res/strings test to enable app_name

Defining a custom res/values/strings.xml prevents nitc from generating a
custom one with the annotation app_name as the resource app_name. This
kept use from having different launcher names for the minimal and
complete test versions.

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

contrib/mnit_test: update the fdroid metadata file

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

contrib/mnit_test: compile 2 versions of mnit_simple for Android

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

contrib/mnit_test: move mnit_simple to contrib/mnit_test

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

contrib/crazy_moles: update Makefile to use the new check-android.sh

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

Merge: New loader policy that features project.ini

This introduce the new loader policy by adapting the existing loader.
The API it still as bad as before but the behavior is expected to be better. Close #1250

The changes impact two decisions made by the loader:

* how to find the project a module belongs to? ie get the root project of an arbitrary file.
* how to find a named module in a project? ie get (and load) the module named `foo` in a mproject object.

The new loader use a recursive directory search for both answers, whereas the older loader only looked at some specific paths.
By the way, the new code use `String::files` and `String::file_stat` (they where unneeded by the previous loader, life was simpler back then). The light FFI used by the bootstrap make this possible.

Here is a detail of the new policy:

* A project is a directory identified with a `project.ini` file.
  Only the existence of the `.ini` file matter, the content is not used yet.
* A group is just a sub-directory of a project; there is no more naming or structural convention (ie hack).
* A module either belongs to a project, or is a standalone module (also called singleton project). This one is not new.

To know the project a module belongs to, its directory is recursively searched up to the `/` directory.
If a project is found, then the module belongs to it.
If no project is found, then the module is a standalone module.

Note: to simplify the migration from the previous loader, a special file `projects.ini` (with a plural) in a directory indicates that sub-directories are projects, even without a `project.ini` file.
These files simplify the PR as only some `projects.ini` files are needed (eg `lib/projects.ini`)

To search for a module in a project, the named module is searched in the directory and subdirectories of the project.

This PR also expose some vocabulary for the end user. Maybe some terms could be improved before the PR is merged. Some propositions:

* rename `project` as `package` (so `package.ini` instead of `project.ini`). Might be non-POLA from people coming from a Java ecosystem.
* rename `group` as `directory`. With the new policy, groups are more transparent than ever and are only here for the programmer to group modules as he prefers.

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

inscape_tool: isolate the test directory since generated files are broken

See #1657

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

platform/android: the root group is no more src, thus no need to `..` from it.

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

tests: update some error messages because of the new loader policy

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

tests: improve loader test with a more complex project1 directory

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

loader: change get_mgroup policy and search for a `project.ini` in parent directories

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

Create empty `projects.ini` and `project.ini` for the new loader policy

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

loader: get_mgroup check and assign the `project.ini` file to projects

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

loader: get_mgroup filter out non-directories

This was implicit because non-directories do not match the heuristics

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

loader: projects might have a ini file associated

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

loader: identify_file distinguishes singleton projects in the verbose message

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

misc/jenkins: improve check-android

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

loader: use `mmodule_paths_by_name` to simplify `search_mmodule_by_name` and `search_module_in_paths`

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

loader: introduce `MGroup::mmodule_paths_by_name` to search for modules

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

loader: scan_group does not recursively scan in case of nested project

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

misc/jenkins: move crazy_moles/check-android.sh to misc/jenkins/

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

Merge: Beefup check contrib

This change make the caller pass the additional `make` rules to execute. So now `check` or `android` can be given to mass build projects that expose such rules.

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

Merge: Fix app.nit Calc example

Partial revert of cbbb5225 because the Android version of Calc need only `android::ui` and not `android`. The default entry point of `android` use `native_app_glue` as base activity/framework, whereas UI apps use `nit_activity`. This caused the apk file to show 2 activities in the launch menu.

Also recent changes (probably) make calling `to_i` and thus `to_n` on empty strings crash. I've updated the calculator to avoid this with a new friendlier behavior in such cases.

I've also updated `to_n` to return 0 on empty strings. This preserve the old behavior, it is a neutral fallback value and it fits with `"".is_numeric == true`. I did not touch `to_i`.

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

Merge: Report check contrib

Thanks to `misc/jenkins/check_contrib.sh`, programs in contrib can be tested for regressions.
Unfortunately, sometime the result of the test is not reported (only printed on screen). This PR update some of these to really report things.

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

Merge: Benchmark strings update

Update on the benchmarks for String, four micro-benchmarks are now available:

* chain_concat: measures the performance of concatenation
* index_bench: measures random access in a String
* iteration_bench: measures the performance of iteration on a string
* substr_bench: measures the performance of substring on a string

The logic for the benches has changed a lot, instead of working locally with the different structures and having a directory with all the variations (more to come in a near future), now a patch directory is included with the benches for Strings, and will work with all the variants one by one on the chosen benchmark.

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

lib/markdown: ignore generated nitmd command

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

lib/json: make build nitcc if required

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

lib/standard: `Text::to_n` returns 0 as failsafe when receiver is empty

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

lib/app: add FIXME to the conditional importation of android::ui

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

examples/calculator: add a debug option

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

examples/calculator: avoid crash from call to `to_n` on empty strings

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

examples/calculator: fix Android version to use nit_activity only

This is a partial revert of cbbb5225. The entrypoint of `android` use
`native_app_glue`, which led to craming 2 activities in the apk file.

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

Merge: Text::is_int bugfix

Fix a bug that caused an assert failed when accessing an empty string in `Text::is_int`

Reported by @xymus with PR #1649

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

check_contrib: additional rules are given by arguments.

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

check_contrib: also check lib

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

Merge: GitHub nitlang

Migrate references from https://github.com/privat/nit to https://github.com/nitlang/nit

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

various lib and contrib doc: points to new github repo

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

lib/github: use new github repo nitlang/nit in doc and tests

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

nitrpg: use new github repo

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

github_merge: use new github repo

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

fdroid: update metadata with new github repo

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

Makefile: link generated doc with new github repo

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

lib/standard/fixed_ints: Added extra check on is_int to avoid crashing on indexed access

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

benchs/strings: Makefile update

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

gitattributes: Stop watching for whitespaces in patch files

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

benchmarks/string: Added variants of lib to benchmarks

* linear_substring: old version of String::substring
* buffered_ns: Perl-like over-allocation when concatenating Strings

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

Merge: objcwrapper: wrapper generator to access Objective-C services from Nit

This PR introduces *objcwrapper*, a wrapper generator to access Objective-C services from Nit. At this point, *objcwrapper* can parse large Objective-C header files from Apple, and generate valid Nit code to wrap simple classes.

This PR contains the work of @Tagachi (as the first commit), some of my refactoring and new features. Because of this, there is some rewrite between commits. I chose to keep them as-is because it may be useful to debug regressions in the future. Actually, some of the refactoring remove a few features (such as super class declaration parsing), they will be brought back in future PRs.

This is a good base for future work but also a work in progress. Please restrain comments to structural and algorithmic problems, the doc and manual will be completed with the missing features.

Working features:
- [x] Generate a Nit extern class for each Objective-C `@interface` block.
- [x] Generate Nit extern methods to wrap Objective-C methods (`- `).
- [x] Generate getters for attributes `@property`.
- [x] Basic type conversion from Objective-C to Nit.
- [x] Partial detection of supported and unsupported types.

Todo features:
- [ ] Static functions (`+`) as top-level methods.
- [ ] Attribute setter (when not `readonly`).
- [ ] Intro and use an `ObjcType` instead of a `String`.
- [ ] Reproduce class hierarchy.
- [ ] Reuse wrapped classes (serialize the model alongside the wrapper).
- [ ] Deal with incompatible types: function pointers and others.
- [ ] Print a report on unsupported services.
- [ ] Support parsing GNUstep classes.
- [ ] Generate valid Nit code with Apple classes.
- [ ] Merge with *jwrapper*?

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

Merge: lib/code/standard: hash no more divides object_id by 8

Since objects are allocated on 64bit word boundaries, it made scene to remove the three 000 lower bits and increase the entropy of the hashcode.
Unfortunately, primitive objects like low Int values or Bool have a object_id lower than 8. shifting the 3 last bits was thus a bad idea.

Therefore, programs that used Int or Bool as keys (or part of keys) did have a lot of hash collisions.
For instance, before, for lib/ai/examples/puzzle.nit:

* number of collisions: 525428 (84.49%)
* average length of collisions: 12.44

After:

* number of collisions: 256223 (41.20%)
* average length of collisions: 3.16

The change have a limiting effect on programs that mainly use standard objects as keys.

Before, for nitc:

* number of collisions: 661715 (16.18%)
* average length of collisions: 2.24

After:

* number of collisions: 711614 (17.40%)
* average length of collisions: 2.25

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

nitiwiki: update results and report test error

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

contrib/objcwrapper: write import only once per file

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

contrib/objcwrapper: refactor services and update style of CodeGenerator

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

contrib/objcwrapper: revamp the AST visitor

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

contrib/objcwrapper: rename a few AST nodes to 'parameter'

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

contrib/objcwrapper: add the -o option to specify the output file

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

contrib/objcwrapper: intro basic framework for command line options

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

contrib/objcwrapper: refactor `init_with_alloc` as an attribute

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

contrib/objcwrapper: merge some type-related tokens in grammar

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

contrib/objcwrapper: move header_static to local folders

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

contrib/objcwrapper: add a new line at the end of the generated file

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

contrib/objcwrapper: report parsing errors

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

contrib/objcwrapper: add some doc and license

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

contrib/objcwrapper: add the check rule with a few tests

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

contrib/objcwrapper: add git structure and Makefile

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

contrib/objcwrapper: intro objcwrapper base code

Signed-off-by: Arthur Delamare <arthur.delamare@viacesi.fr>
Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>