lib: migrate nitunits to annotations

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

json: fixme interpreter bug in constructors with return (#2546)

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

serialization: move StrictHashMap related services

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

serialization: move up deser. safety services from `json`

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

Merge: nitc: loader skip dot (hidden) directory

This will improve the loading time when there is a very
deep .git directory around.

Pull-Request: #2534

Merge: gamnit audio: control sound channels and don't crash if there's no sound card

Desktop clients can now control playing sounds with the new services from SDL2 mixer. The equivalent was already implemented for Android but not GNU/Linux. The new services still need to be fully exposed in the `app::audio` API.

This PR also removes the requirement to have a working sound card to launch a gamnit game.

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

Merge: json: remove the old nitcc parser

Improve performances of the `json` package by removing the old nitcc parser to use only the custom string parser. It is faster at runtime and the nitcc parser classes were included with the JSON serialization services, slowing down compilation time even when not used.

The nitcc parser was still used only by a few clients, the only differences for the clients should be the error message, conflicts on the `Location` class, and improved performances.

It may be hard to see in the diff but the only differences in the `json` package are: the removal of the nitcc parser, move the string_parser in `static` (they are highly coupled), and make a few services private.

Note that the JSON nitcc grammar is still available in the nitcc project folder.

Warning: Until there's a service like `full_class_name`, deserializing classes with a name conflict is broken. I don't think that the 2-3 cases in the PR deserialize the generated JSON, so it should be OK for now.

Pull-Request: #2540

Merge: dom: fix two possible out of bounds string accesses

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

Merge: lib/markdown: fix crash of the parser when parsing unfinished refs

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

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

Merge: lib/mongodb: fix database_names test

Pull-Request: #2545

gamnit: don't crash if there is no sound

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

linux::audio: extract `play_channel`

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

sdl2: add more `mixer` services

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

serialization: move up type name manipulation services out of `json`

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

json: test deser. from plain JSON and fix typo

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

gamnit: intro basic shadow mapping

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

tests: udate more tests testing or depending on the JSON parser

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

model & neo: fix ambiguity in the use of Location

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

neo_doxygen: fix a few "type repetition" warnings

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

neo_doxygen: remove ambiguity in the use of Location

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

json: update expected error in unit test

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

json: remove the old nitcc parser

And move the content of `string_parser` to `static` because of high
coupling.

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

lib/mongodb: fix database_names test

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

lib/markdown: fix crash of the parser when parsing unfinished refs

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

gamnit: add camera parameter to `draw` method

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

gamnit: intro UniformInt

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

gamnit: cull back faces

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

gamnit: fix 3D objects in dynamic resolution

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

dom: fix two possible out of bounds string accesses

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

git: restrict -diff attributes to specific files

Fix an issue where all `parser.nit` files didn't show a diff.
It wrongly caught lib/dom/parser.nit.

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

nitc: serialization phase use the qualified class name in code gen

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

bytes: tweak doc to put forward the copy behavior of the slices

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

gamnit: print current config and list extensions

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

glesv2 & gamnit: fix typos and minor issues

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

glesv2: intro glGetString

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

nitdoc: set icons for each kind of cards

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

nitweb: display defs tab for mpackages and mgroups

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

nitdoc: api returns defs for packages and groups

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

nitweb: do not show inheritance data in doc tab

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

nitweb: show parents and children in inheritance tab

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

nitweb: serialize POSetElement to JSON

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

nitweb: remove doc tab from entity card

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

nitweb: do not return full html doc in entity json

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

nitweb: frontend load full mdoc

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

nitweb: api serves full mdoc

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

lib/popcorn: migrate nitunits to annotations

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

contrib: migrate nitunits to annotations

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

lib/core: remove useless test_suite

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

nitunit: update tests

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

lib/core: introduce `test` annotation

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

nitunit: use `intern` constructor

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

nitunit: use `before`, `after` annotations for test cases

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

nitunit: use `before_all`, `after_all` annotations for propdefs in Sys

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

nitunit: use `test` annotation for propdefs

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

nitunit: use `test` annotation for classdefs

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

nitunit: use `test` annotation for modules

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

loader: use `test` annotations for modules

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

parse_annotations: parse nitunit related annotations

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

model: introduce nitunit annotations in mentities

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

json: deser. chars serialized as strings and bytes as ints

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

json: support `Byte` in a similar way to `Char`

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

serialization: fix typo and warning

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

json: avoid repeating if condition

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

json: don't reserve an id when not using it

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

nitc: loader skip dot (hidden) directory

This will improve the loading time when there is a very
deep .git directory around.

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

nitmetrics: dont crash if nclassdef.mclassdef is null

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

nitmetrics: backport some code of subtype test

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

Merge: Nitsmell : Adding new code smells and print console updated

Add :
- New "no abstract implementation" code smell
- Toolcontext option to print one by one a detected codesmell
Update :
- Print console improve
- Feature envy codesmell prepose a move help

Pull-Request: #2514
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>

Merge: serialization: safe formal types, fix warnings in generated code and more

* Use `GetName` to better restrict the accepted static type for attributes with formal types. This should complete the protection against the injection of unexpected types through deserialization (reported by @ppepos).
* Update `nitserial` to intrude import modules with private serializable classes, instead of skipping them. Serializing private classes from the lib should probably be avoided, but at least now we support them and we can see them.
* Fix the deserialization of maps from JSON with metadata and cycles. A difference in the deserialization order could cause a reference to be read before the referenced object. This issue may still be caused by versionning, so a foolproof solution should probably be added to the JSON deserializer.
* Fix a warning in generated code, it was generated for each `serialize` class (quite a lot). It caused 25 warnings just in lib/github/api.nit.
* Implement serialization for the private classes of `core::queue` and don't crash on deserialization errors in `a_star`.

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

Nitsmell : Adding new code smells and print console updated

Add :
- New "no abstract implementation" code smell
- Toolcontext option to print one by one a detected codesmell
Update :
- Print console improve
- Feature envy codesmell prepose a move help

Signed-off-by: Florian Deljarry <deljarry.florian@gmail.com>

Merge: gamnit: misc fixes

Fix various bugs and issues related to gamnit :

* Expose the previously private class `SpriteSet` for clients to access `time` and set `time_mod` (as intended).
* Fix two bugs with animations: a div by zero in the shader that hid non-animated sprites on Android (a div by zero never crashes, but the return value is unspecified in OpenGL ES 2.0), and an array out of bounds error when an animation has a single frame.
* Cache `GamnitDisplay::aspect_ratio` as it appears to be a performance bottleneck.
* Fix the virtual gamepad not being interactive in Asteronits.
* `UICamera::camera_to_ui` returns a `Point3d` that can be used directly as a `Sprite::center`.

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

asteronits: fix virtual gamepad

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

gamnit: don't remap sprites removed from a set after their modification

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

gamnit: fix div by zero when there is no animation

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

gamnit: fix animations with a single frame

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

gamnit: make `SpriteSet` public so clients can use its services

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

gamnit: `UICamera::camera_to_ui` returns a 3D point

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

gamnit: cache `GamnitDisplay::aspect_ratio`

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

lib: misc typos

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

a_star: don't crash on deserialization errors and limit static types

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

serialization: implement serialization for classes of `core::queue`

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

tests: add formal types to serialization tests

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

frontend: don't deserialization code with warnings

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

frontend: don't mention JSON in general deserialization error

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

frontend: safer deserialization of parameter and virtual types

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

json: standardize error messages

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

json: deserialize Maps keys first, in the same order as they are written

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

json: use GetName to restrict deserialized formal types

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

serialization: intro `AttributeMissingError`

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

serialization: update some doc

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

nitserial: intrude import modules when needed

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

gen_nit: accept complete import declarations

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

Merge: typing: new warning `useless-truism` to catch trainees.

Warning on `if a == true then` constructions.

Pull-Request: #2520
Reviewed-by: Jean-Christophe Beaupré <jcbrinfo.public@gmail.com>

Merge: gamnit: scale text, add custom characters and create links with `BMFont`

This PR adds more customization options when drawing text using `BMFont`:

* Change the size of the text with a simple attribute `TextSprites::scale`.
* Insert artificial chars to a `BMFont` using the more permissive `BMFontChar`. This way, one can set custom textures for a letter, an emojis or any character from the private use areas.
* Add "links" to the text using Markdown-like syntax (`[my link]` or `[pretty name](internal name)`). Note that the links are not clickable by themselves. However, the client code can access the sprites representing the link to change their style or to get their position. This will work well with a rumored future feature making sprites clickable...

Notice the colored links and the smaller text box in the following screenshot of `fonts_showcase`:

---

Once again, these new features are not implemented for `TileSetFont`. I'm considering keeping the API of `TileSetFont` but replacing its implementation by a `BMFont` for it to gets all the new shiny features.

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

gamnit: add new features to the fonts_showcase example

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

gamnit: add markdown-like links to `TextSprites` with `BMFont`

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

lib&contrib: remove some useless truisms.

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

Merge: model: group parameter types in `MSignature::to_s`

Modify `MSignature::to_s` to make its output a shorter valid Nit signature. This helps readability and shortens error messages allowing them to fit more easily on a single line or within a text editor.

Example program:
~~~nit
import geometry
var b = new Box3d[Float] # <- error
~~~

Previous error message:
~~~
Error: expected 6 argument(s) for `init(left: Float, right: Float, top: Float, bottom: Float, front: Float, back: Float)`; got 0. See introduction at `core::Object::init`.
~~~

New error message:
~~~
Error: expected 6 argument(s) for `init(left, right, top, bottom, front, back: Float)`; got 0. See introduction at `core::Object::init`.
~~~

Pull-Request: #2529

Merge: quadtree: use an array, fix warnings and standardize the doc

Replace the 4 attributes by an array of all the children. This new structure lightens the code and allows to fix 65 `method call on a nullable receiver` warnings. As this is already a significant refactor, I avoided modifying further the implementations to avoid breaking things.

Pull-Request: #2528

Merge: gamnit: control drawing order

When two sprites are overlapped, a non-opaque pixel drawn before the sprite behind it may be blended with the wrong color. The only solution is to control the draw order to ensure that sprites in the back are drawn first. However, this is hard to automate when grouping sprites in buffers to optimize the drawing of a very large number of sprites (as gamnit does). It requires to break off groups in layers and this may be very costly if the sprites move on the Z axis.

The solution it to let the programmer choose a draw order only when needed. This works pretty well in 2D games where some objects stay on the same relative depth layer.

> Example artifacts on the left, and the fixed result (with a `sprite.draw_order = 1`) on the right:
>
>  

This PR introduces the attribute `Sprite::draw_order` that can be set to a higher value (default at 0) for the sprite to be drawn later. In general, sprites with a non-opaque `texture` and sprites closer to the camera should have a higher value to be drawn last.

Sprites sharing a `draw_order` are drawn in the same pass. The sprite to sprite draw order is undefined and may change when adding and removing sprites, or changing their attributes. A future improvement could sort sprites in the same group automatically if it becomes an issue.

Note that changing the `draw_order` may have a negative performance impact if there are many different `draw_order` values across many sprites. This may break the grouping optimization by creating too many small groups.

---

I plan on adding the draw order to `gamnit::depth` actors too. This will be useful for games than mix 2D with 3D. However, I would have to apply a different strategy for pure 3D games, implementing two main passes, one for opaque objects and one for non-opaque objects.

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