shibuqam: we can use `string_arg` directly

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

nitcorn: fix decoding GET args from percent encoding

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

Merge: iOS: complete support for the UI API

This PR complete the iOS support of the _app.nit_ UI API and fixes a few bugs:

* Fix `ListLayout` (the scrolling view) so its content fits in the horizontal limits of the screen. (The solution was the "-0-" in the constraint string!)
* Fix detecting when the user taps the system "back" button to correctly resume the previous window.
* Implement `size=` and `align=` on `TextInput` and `Button`. There is some code duplication because the Objective-C properties are not the same.
* Add `TextInput::placeholder=`, the placeholder is often used to identify a text field without using a label on iOS.
* Intro more color related services to extern classes, including a constructor to get the default light gray color used in the background of system menus.
* Bypass the Unicode validity checks so that Unicode characters are left as is, a workaround for #1945. This will cause problems on invalid Unicode strings, but it does work for valid Unicode strings.
* As a bonus, update the resolutions of the icons generated by `inkscape_tools`. Dropping support for older iOS versions and adding support for newer versions.

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

Merge: android: click event on button release, better errors and safer `JavaString::to_s`

Some small tweaks to the Android support:

* `JavaString::to_s` used to crash (at the JNI level) if the associated Java string was null. This was a counterintuitive behavior when the Nit object is non-nullable. This PR prevents this error by returning something similar to the default `Object::to_s` when the underlying Java value is null (for `JavaString` and others). This makes it crash proof, but one should still check if `JavaString::is_java_null` before using the value returned by `to_s` when an actual string is expected.

* Some Java exceptions raised by the Android implementation of `http_get` do not have a message. This PR looks through the causes of such exceptions to find the first available message.

* Also, fix `on_click & ButtonPressEvent` to be raised only when a button is released (instead of at the initial touch).

Pull-Request: #2319
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>
Reviewed-by: Ait younes Mehdi Adel <overpex@gmail.com>

inkscape_tools: update the format of icons generated of iOS

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

ios: don't force spacing between elements of layouts

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

ios: intro services for UIColor, UILabel & UITextField

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

ios: workaround for unicode issue #1945

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

ios: intro custom view controller and catch back button press

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

ios: fix ListLayout to fit horizontally

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

ios: redirect prints to NSLog (printf is no longer an output of the simulator)

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

ios: duplicate full implementation of `size= & align=` in TextInput

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

ios: Button relay `size= & align=` to its `title_label`

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

ios: refactor `Label::size= & align=` services implementation

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

Merge: iOS: fix compilation problems, warnings and the Benitlux ios-release rule

General fixes for the iOS support:
* Report errors on `rmdir`, this depends on the fix #2309.
* Fix release mode compilation, the path to the binary was wrong.
* Fix the ios-release rule of Benitlux to use the release mode, contact the right server, and keep the generated files to use from XCode.
* Finally silence the normal output of `xcodebuild`. This looks like a new feature of XCode 8!
* Fixed a few warnings raised by clang.

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

java: don't crash on `to_s` when not null in Nit (even if null in Java)

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

android: trigger click event on button release

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

android::http_request: fix exception handling

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

tests: skip postgres tests on macOS

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

Merge: lib/json: unify the writing API using serialization only

Some context first, before this PR, there were 2 API to write JSON:
* The module `static` provided `to_json` and `to_pretty_json` which supported writing only basic Nit types associated with JSON types and objects with a manually implemented `to_json`.
* The module `serialization` provides `serialize_to_json` which supports all `Serializable` types, a superset of the JSON types, and where standard Nit objects are serialized to JSON objects by generated code.

The advantages of the `serialization` API are:
* Objects written to JSON with some metadata (mainly the name of the Nit type) can be deserialized back to Nit objects.
* The boilerplate code to write the attributes is generated, usually a single `is serialize` after the module declarations does the trick. (`core_serialize_to`, the equivalent of `to_json`, doesn't have to be implemented in each classes, the generated version is usually enough.)
* Implementing `core_serialize_to` (or leaving it to the generated code) instead of `to_json` makes the object compatible with both the JSON and binary serialization services.
* In general, the `serialization` API allows to easily write Nit objects to JSON and build JSON objects using `Map` instances:
~~~
import json::serialization

class Person
    serialize

    var name: String
    var year_of_birth: Int
end

var bob = new Person("Bob", 1986)
assert bob.serialize_to_json(pretty=true, plain=true) == """
{
    "name": "Bob",
    "year_of_birth": 1986
}"""

var charlie = new Map[String, nullable Serializable]
charlie["name"] = "Charlie"
charlie["year_of_birth"] = 1968
assert charlie.serialize_to_json(pretty=true, plain=true) == """
{
    "name": "Charlie",
    "year_of_birth": 1968
}"""
~~~

So this PR drops the `static` writing API, and replaces its services (`to_json` and `to_pretty_json`) by aliases in `serialization`. Some subclasses have been updated to implement the recursive `serialize_to` instead of the `to_json`, the changes are minimal to avoid breaking any software, but they are not optimal. The result is a single JSON writing API!

The generated JSON should not have changed much (maybe a bit less or more white spaces), except for some error messages which were translated to invalid JSON and are now serialized to a valid JSON object.

I expect some tests to fail as this is a big change, and clients of the `json::static` writing services (@morriar) should probably double check my changes to their code.

In the future, I'm thinking of dropping the name `serialize_to_json` and `to_pretty_json` and move the optionnal parameters to `to_json`. The JSON module also really needs refactoring at this point.

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

Merge: nitcorn: possible fix for the bug on input with unicode

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

tests: iOS app have one more file

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

nitcorn: fix input buffer containing unicode

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

benitlux: fix ios-release rule and keep the compiled files for XCode GUI

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

lib: fix clang warnings in md5 and pthreads

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

src: `exec_and_check` print to stderr

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

nitc iOS: quiet output of `xcodebuild`

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

lib & contrib: update imports

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

lib/json: split the serialization module, replace package entrypoint & update doc

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

Merge: core: fix rmdir not reporting some errors

Fix `rmdir` to raise an error on failure to remove a folder, instead of on success. Also fix error management in recursive calls where some errors could be lost.

The name of the attribute `Path::last_error` is misleading with `rmdir` as it stores the first error of the last call, and it is set to `null` if there is no error (overwriting the true last error), so it very rarely contains the actual last error. Note that, `Path::last_error` has a different logic than `Stream::last_error` which does keep the last error and does not reset it. In the future, I would suggest to rename `Path::last_error` to `error` to avoid confusion or to use an array that is cleared at the beginning of each method (as it is currently done with `Path::last_error`).

Pull-Request: #2309
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

neo_doxygen: fix some typos and warnings

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

neo_doxygen: remove duplicate tests (copied from docunits)

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

neo4j: update expected test results with full serialized objects

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

refund: force last line break and update expected errors

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

tests: update expected results of serialization & json tests

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

json::serialization: inline arrays

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

json::serialization: 2 less spaces in non-pretty mode

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

refund & nitrpg: rename 2 conflictual to_json to to_json_object

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

tests: update deserialization expected error

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

*: update redefs of `to_json`

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

json::serialization: add an example for building a JSON object from a Map

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

json::serialization: add compatibility methods to_json and to_pretty_json

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

json::static: all Jsonable are Serializable

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

json::static: reword module doc

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

lib/json & nitcc_runtime: make some classes serializable

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

lib/json: remove duplicated JSON writing services from static

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

lib/json: bring back string to JSON service to `json::serialization`

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

core::file: fix error management in recursive calls to `rmdir`

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

nitiwiki: update test result to reflect correclty cleaned directories

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

nitc iOS: fix moving binary in release compilation mode

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

nitc iOS: report rmdir errors and quit

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

core::file: add docunit for expected behavior on deleting existing folders

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

core::file: fix `rmdir` to raise an error on failure to remove a folder

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

Merge: StatementRow::map

I thought this was lacking from the API, so now when dealing with a StatementRow, one can get a map from column name to column value.

NOTE: This is not lazy since Sqlite re-uses the StatementRow for each row returned when a select statement is sent to the database.

Pull-Request: #2308
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>

tests: Update `test_sqlite3_nity` to include `StatementRow::map`

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

lib/sqlite: Added `map` method to `StatementRow`

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

Merge: gamnit: fix field of view issues

Erroneous code at creation of the perspective matrix caused a wider than requested frustum and field of view. This caused issues in other gamnit code:

* The service `camera_to_world` had to multiply the result by 1.72 to correct the deformation.
* In `asteronits`, the world was not filling the screen, only part of it.
* In general, it created a cool but not desired wide angle lens / fisheye effect.

After the fix, some clients might have to widen the requested visible area, usually set with `reset_height`. We may have to lower the default `field_of_view_y` as it is a bit high.

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

action_nitro: see more world units on screen

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

gamnit: the magic number has been fixed by the fovy

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

gamnit: fix setting camera field of view

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

Merge: nitweb: render wikilinks

This PR allows nitweb to render wikilinks from README.md files.

Here the demo with the lib/core/README.md file: http://nitweb.moz-code.org/doc/core

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

nitweb: replace default md_processor so nitweb can render README files

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

nitweb: move md_processor to config

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

src/doc_down: does not pre escape nit source code

Fixes a problem of rendering when the nitunit contains `"`

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

src/doc_down: make md processors writable

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

Merge: nitweb: use index for quick search

This PR use the `model_index` API to provide better quicksearch in nitweb.

Demo: http://nitweb.moz-code.org/

Will maybe close a shitstorm of quicksearch related issues but I'm to lazy to check the numbers...

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

Merge: nitweb: better namespaces

Before this PR, the namespace composition (adding html things to the MEntity::full_name) was handled by Angular. Namespaces are a pain in the nit to generate, even more when you don't have access to the model... So the solution was to delegate that work to the API.

With this PR, nitweb attach a Namespace object to each MEntity so Angular knows how to render it.
The idea is pretty simple, a namespace is an array of either:

* a string for "::", "$" ...
* an mentity reference like `Array`
* another namespace for recursive definitions

Demo is here: http://nitweb.moz-code.org/

Some examples:

* `core`: http://nitweb.moz-code.org/doc/core
* `core>`: http://nitweb.moz-code.org/doc/core>
* `core::core`: http://nitweb.moz-code.org/doc/core::core
* `core::Array`: http://nitweb.moz-code.org/doc/core::Array
* `core::Array::from`: http://nitweb.moz-code.org/doc/core::Array::from

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

Merge: frontend: make the warning on unavailable glslangValidator an advice

Changes the following warning to an advice and raise it at each use of the annotations (instead of only once).

~~~
/nit/lib/gamnit/flat.nit:355,9--26: Warning: program `glslangValidator` not in PATH, cannot validate this shader. (glslvalidator)
""" @ glsl_vertex_shader
~~~

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

Merge: app::audio: remove deprecated services and other minor improvements

This PR removes the old `load_sound` and `load_music` services, they were replaced by lazy loading services a while back. The new services are more in line with the other asset loading services from `gamnit`.

There are also other general but minor improvements: doc updates, better names for a param and an attribute, less warnings, less casts, more consistent error messages and easier debugging.

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

Merge: typing: Fix minor mistakes in documentation

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

Pull-Request: #2301

Merge: json::serialization: don't raise errors on missing attributes with default values

This PR fits in the ongoing work to improve deserializing from plain JSON objects.

The JSON deserialization engine no longer raises errors on missing attributes when a default value is available. Attributes may be missing because the JSON object come from a third-party API or
when loading serialized data from a previous version of the software. Default values include simple default values (`var x = 4`), lazy attributes and nullable types (which are set to `null`). This does not yet include `optional` attributes, more work would be needed.

The test/example can be activated when #2296 is fixed.

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

Merge: nitweb: light responses

Nitweb was returning insane amount of data (~2mo of json by request). The new version (http://nitweb.moz-code.org/) is a little more perf.

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

nitweb/ng: enhance search results display

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

nitweb/ng: clean card css

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

nitweb: use model index

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

nitweb: move view to config

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

nitweb: autoload focused linearization code

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

nitweb: show namespace in frontend

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

nitweb: add `namespace` key to all mentities json

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

model: fix some warnings

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

nitweb: fix linearization responses

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

nitweb: use model_json `full_json`

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

src/model: split model json into short and full json

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

tests: udate JSON tests

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

json::serialization: document reading from JSON with missing attributes

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

frontend/serialization: missing nullable attribute are set to null

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

serialization: don't throw missing attribute error if there is a default value

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

frontend/serialization: leave missing attributes to their default value

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

json::serialization: engines report when an attributes is missing

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

frontend/serialization: remove generated documentation

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

modelize: precise in `has_value` doc the link with `is_lazy`

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

typing: Fix minor mistakes in documentation

Signed-off-by: Jean-Christophe Beaupré <jcbrinfo@users.noreply.github.com>

android::audio: remove indirection to sounds.add

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

android::audio: revamp param and doc of `load_name`

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

app::audio: rename `name` to `path`

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

app::audio: update doc

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

android::audio: print errors at loading sounds

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

frontend: always raise advice about missing glslangValidator

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

frontend: make the warning on missing `glslangValidator` an advice

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

app::audio: drop the old load_sound and load_music

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