modelbuilder_base: Refactor the signature of `resolve_mtype*`

Only `modelize_mclass` require support for partial context. Furthermore, in
a future PR, it may need to resolve types just before modeling the class
definition (in a place where the `MClass` is available and must be used).

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

modelbuilder_base: Rename some local variables

Avoid a name conflict in future commits.

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

typing: Make `TypeVisitor::mmodule` implicit

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

typing: Make `TypeVisitor::anchor` non-nullable

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

typing: Make `TypeVisitor::mclassdef` non-nullable

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

typing: Make `TypeVisitor::mpropdef` non-nullable

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

typing: Remove duplicate assignment to `TypeVisitor::mpropdef`

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

modelize_class: Fix spacing

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

Merge: geometry: implement 3D `dist` and improve API

The main change in this PR is the reimplementation of `Point::dist` and `dist2` to support 3D points, including comparing 2D and 3D points. The double dispatch logic is a bit complex but it works with any receiver.

This PR also tweaks existing services to improve the API and doc:
* Make all attributes of `Point` optional for gradual creation/space allocation. In practice, points are often created with a default value, as a kind of buffer, or with the Z axis at 0.
* Rework `BoxedArray` as a subclass of `Array` to inherit its services (incl. `remove`) while using less code.
* Move the custom `to_s` from the concrete boxes to the interfaces as it is very useful when debugging.

Pull-Request: #2462

Merge: lib/popcorn: introduce `AppConfig::app_hostname`

Introduce `app_hostname` in popcorn AppConfig.

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

Pull-Request: #2464

Merge: gamnit & sdl2: implement app::audio using SDL2 mixer

Intro a decent implementation of `app::audio` for desktop computers using SDL2 mixer. It supports WAV, MP3, OGG and more. The underlying SDL2 mixer bindings are minimal, they offer only the required features, more can be added as needed.

Clients of the gamnit framework will need to `apt-get install libsdl2-mixer-dev`.

Pull-Request: #2466

geometry: implement `Point::dist` for 3D points

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

geometry: make `BoxedArray` a subclass of `Array`

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

geometry: make the attributes of points optional

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

geometry: move up custom `to_s` to the interfaces

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

gamnit: implement app::audio for desktop using SDL2 mixer

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

sdl2: intro minimal bindings to SDL2 mixer

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

audio: move some services between the abstraction and the Android layers

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

lib/popcorn: introduce `AppConfig::app_hostname`

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

Merge: Popcorn templates

# Template rendering for popcorn

## Basic templates

Use TemplateString to render really basic templates that just need macro
replacements.

Example:
~~~nit
class TemplateStringHandler
        super Handler
redef fun get(req, res) do
# Values to add in the template
var values = new HashMap[String, String]
values["USER"] = "Morriar"
values["MYSITE"] = "My super website"

# Render it with a shortcut
res.template_string("""
<h1>Hello %USER%!</h1>
<p>Welcome to %MYSITE%.</p>
""", values)
end
 end
 ~~~

 For larger templates, you can also use external files (makes your Nit code cleaner):
 ~~~nit
 class TemplateFileHandler
super Handler

redef fun get(req, res) do
# Values to add in the template
var values = new HashMap[String, String]
values["USER"] = "Morriar"
values["MYSITE"] = "My super website"

# Render it from an external file
res.template_file("example_template.tpl", values)
end
 end
 ~~~

 ## Using pug templates

 Pug is a templating format provided by the external command `pug`.
 For complex templates that need conditional or loop statements, pug can be a solution.

 See the pug syntax here: https://pugjs.org/api/getting-started.html

 ~~~nit
 class PugFileHandler
super Handler

redef fun get(req, res) do
# Values to add in the template
var json = new JsonObject
json["user"] = "Morriar"
json["mysite"] = "My super website"

# Render it from an external file
res.pug_file("example_template.pug", json)
end
 end
 ~~~

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

Merge: coloring: Only generate strictly positive IDs

This will allow to reserve negative IDs (including 0) for special purposes.

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

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

Merge: model: Do not assume enums to be final

An upcoming feature will destroy this assumption.

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

Pull-Request: #2458

Merge: gamnit: intro support for Angel Code bitmap font

The BMFont format supports packed textures, varying advance per character and even kernings. It can be generated by a number of tools, including:
* BMFont, free software Windows app, http://www.angelcode.com/products/bmfont/
* Littera, a web app, http://kvazars.com/littera/

This PR also generalizes the `Font` and `TextSprites` concepts, previously limited to tile set bitmap fonts. And add a new shortcut service to the excellent DOM parser used to parse the XML description.

The result is much nicer than what was previously available, you can see it in a preview of my latest project:

The parsing method crash easily on malformatted font description file. However, since the description file should be generated, loaded at game start and not often modified, all expected crash errors should occur in dev only. Errors that can happen in production, like a truncated file or a misplaced file, should be detected by the basic sanity checks.

If an error occurs while loading a font, there is currently no automatic fallback and it will probably lead to an unusable game. It could be remedied by adding a default font, similar to the default material and mesh. It would be very useful, even at the beginning of any projects, but there is some technical limitations to implementing it as of now.

In the future, I plan on moving up more services from `TileSetFont` and add text alignment features. The `flat` module needs to be split in a group to include more services, including fonts.

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

gamnit: intro BMFont parser and drawing services

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

gamnit: extract general Font from TileSetFont

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

dom: intro `XMLTag::attributes_to_map` mapping `XMLStringAttr` to their value

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

tests: Update error messages

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

model: Do not assume enums to be final

An upcoming feature will destroy this assumption.

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

tests: Do not rely on the implicit fixing of type bounds

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

Merge: model: Fix minor documentation mistakes

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

Pull-Request: #2451

Merge: nitvm: Factorize the loading of superclasses

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

Pull-Request: #2452

Merge: parser_nodes: Fix documentation

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

Pull-Request: #2453

Merge: rosettacode: Fix spelling mistakes

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

Pull-Request: #2454

Merge: rapid_type_analysis: Fix a spelling mistake

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

Pull-Request: #2455

dom: return a more precise `Array[XMLTag]` as the doc already said

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

metrics_base: Do not rely on the implicit fixing of type bounds

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

opts: Do not rely on the implicit fixing of type bounds

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

coloring: Only generate strictly positive IDs

This will allow to reserve negative IDs (including 0) for special purposes.

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

coloring: Fix documentation formatting

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

rapid_type_analysis: Fix a spelling mistake

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

rosettacode: Fix spelling mistakes

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

parser_nodes: Fix documentation

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

model: Fix minor documentation mistakes

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

Merge: Popcorn: clean tests

This PR does two things:
* clean the lib by moving all the tests to `tests/` directories, easier to see what is important, great visual improvement for the examples.
* use a lot of imagination to avoid the `class name X already used`. It mostly replaces Short&Cool names by LooooonnggggandAndBooooringNameForSoSimpleHandler. No more warnings when make checking.

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

Merge: MultiHashMap: Provide the converse of `add_one`

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

Pull-Request: #2446

nitvm: Factorize the loading of superclasses

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

lib/popcorn: add documentation for pop_templates

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

lib/popcorn: add pug rendering

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

lib/popcorn: introduce basic templates

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

Merge: gamnit: add documentation and standardize the API

Improve the documentation for the central services of gamnit, mainly for entities visible from gamnit.org. Also, to improve the API, remove the `Gamnit` from some class name and move a few named constuctors to factories or their own class.

Pull-Request: #2422
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>
Reviewed-by: Jean Privat <jean@pryen.org>

Merge: rosettacode: Add integer_overflow rosetta code exemple

Signed-off-by: PatrickBlanchette <PatrickBlanchette@users.noreply.github.com>

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

Merge: Popcorn: more validators

Add two more validators:
* URLField that ensures a field contains a valid URL (you want to redefine that regexp).
* UniqField stubs for uniq emails, logins, ids validation.

How to validate uniq field:

~~~nit
class UniqEmailField
super UniqField

var users: UsersRepository

redef fun check_unicity(v, field, val) do
var user = users.find_by_email(val)
if user != null then
v.validation.add_error(field, "Email `{val}` already used")
return false
end
return true
end
end
~~~

Pull-Request: #2435

Merge: lib/popcorn: introduce tasks

This PR is based on #2436

## Popcorn threaded tasks

 Tasks allow you to execute code in another thread than the app listening loop.
 Useful when you want to run some tasks periodically.

 Let's say you want to purge the `downloads/` directory of your app every hour:

~~~nit
 class PurgeTask
super PopTask

var dir: String

redef fun main do
loop
dir.rmdir
3600.sleep
end
end
 end

 var app = new App

 # Register a new task
 app.register_task(new PurgeTask("downloads/"))

 # Add your handlers
 # app.use('/', new MyHandler)

 # Run the tasks
 app.run_tasks

 # Start the app
 app.listen("0.0.0.0", 3000)
~~~

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

Merge: model: Specify `MClassDef`s’ order once

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

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

Merge: core: fix dynamic error on the input type in `Text::replace`

The previous use of the virtual `SELFTYPE` on a parameter of `replace` was too restrictive and caused a dynamic crash when replacing something in a  `String` by a `FlatBuffer` (for example). This PR uses the more general `Text` as the parameter type, which is already supported by the existing implementation.

The minimal case is not the most useful as doc but I've added it as a unit test anyway:
~~~
var t: Text = "hello"
assert t.replace("hello", new FlatBuffer) == ""
~~~

Pull-Request: #2447

Merge: lib/core/stream: fix typo in doc

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

Pull-Request: #2448

lib/popcorn: fix file inclusion for Jenkins tests

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

lib/popcorn: add `test_path` shortcut in pop_tests

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

lib/popcorn: lot of class renaming in examples and tests to avoid the `class X already defined in...` warning

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

lib/popcorn: move examples tests to nice and tidy tests/ directories.

So the user/reader directly see the important files.

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

lib/popcorn: move lib tests to a nice and tidy tests/ directory

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

lib/popcorn: fix serialization of validation errors

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

lib/popcorn: introduce UniqueField stub validator

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

lib/popcorn: introduce URLField validator

Someone is going to be picky on that re...

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

lib/core/stram: fix typo in doc

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

gamnit and related: improve doc

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

lib/popcorn: introduce popcorn tasks

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

core/text: fix input type in `replace`

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

Merge: nitcorn: fix HttpRequest::int_arg

Before this commit, sending `?n=9.5` to a nitcorn application using `int_arg`
caused the app to crash (see the following to_i).

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

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

Merge: Custom serializer

This is an old attempt to add the possibility to have a custom serializer. The idea is that a same set of objects can be serialized in different ways if different serializers are used.

Most of what is needed to do that is already present; the main contribution of this PR is the example that tries to propose a way to do that. The example focuses on 3 simples needs:

* inject a phantom attribute.
  when serializing, the custom serializer inject new attributes.
* hide a normally serialized attribute.
  when serializing, the custom serializer hides some specific attributes.
* replace a full business object.
  instead of serializing an attribute, the custom serializer use a different representation

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

Merge: lib/core/time: more time related services

Three things here:
* Introduce `Int::sleep` because of my hatred for `1.0.sleep`
* Introduce `get_microtime` because timestamp are better in microseconds
* Introduce `get_millitime` because sometimes, god forbid, we have to deal with Javascript

Also, I'm using `sys/time`, pretty sure I'm doing something that should not be in the core library.

Pull-Request: #2436

Merge: lib/popcorn: replace json_error method by something more useful

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

Pull-Request: #2438

Merge: lib/json: remove Jsonable and replace it by Serializable

Remove the now useless Jsonable class.

Let's see if something is broken.

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

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

Merge: Sha1 is digest

Change receiver of Sha1 methods from String to Text since no service was specific to immutable instances, and add a sha1_is_digest method to Text, to check whether an instance matches the signature of a SHA-1 digest or not.

Pull-Request: #2440

Merge: separate_compiler: Fix a spelling mistake in a comment

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

Pull-Request: #2443

Merge: Makefile: Fix glob patterns

Forgot the call to `$(wildcard)`.

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

Pull-Request: #2444

MultiHashMap: Provide the converse of `add_one`

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

MultiHashMap: Add a test for `add_one`

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

examples: rename Int::sleep from FFI examples to Int::sleep_seconds

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

model: Specify `MClassDef`s’ order once

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

model: Register sorters as private attributes of `MModule`

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

rosettacode: Add integer_overflow rosetta code exemple

Signed-off-by: PatrickBlanchette <PatrickBlanchette@users.noreply.github.com>

Makefile: Fix glob patterns

Forgot the call to `$(wildcard)`.

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

separate_compiler: Fix a spelling mistake in a comment

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

lib/sha1: add Text::is_sha1_digest

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

lib/sha1: replace String by Text for sha1 methods

Since SHA-1 methods might be applied on any kind of Text entity,
we replace the receiver class by Text.

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

tests: update test for nitserial

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

lib/json: remove Jsonable and replace it by Serializable

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

lib/popcorn: replace json_error method by something more useful

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

lib/core/time: also sleep on integers

It always bothered me to write `1.0.sleep`...

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

lib/core/time: introduce get_millitime and get_microtime

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

nitcorn: fix HttpRequest::int_arg

Before this commit, sending `?n=9.5` to a nitcorn application using `int_arg`
caused the app to crash (see the following to_i).

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

Merge: nitc: add option --run to execute after a successful compilation.

When the naive interpreter is tooo slooooow but you are too lazy to run the binary yourself.

Pull-Request: #2432

Merge: Nitweb: api for code highlighter and nitpicker

Only the low-level JSON-REST API  is provided. Someone should have to do the rendering.

ping @Morriar

Pull-Request: #2198

Merge: gamnit: fix nitunit compatibility

Unit tests in the gamnit module were not executed because Sys::setup exited right away when detecting NIT_TESTING. It affected all gamnit games. This PR check is a platform is bound at the beginning of `Sys::main`, allowing unit tests to run (whether they redef `main` or not).

The check is made possible by the new `bound_platform` which tells what platform is currently bound. This value can be useful in some other cases.

This is an alternative to #2394.

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

Merge: core & gamnit: basic support for the characters nbsp, pld and plu

Detect non-breaking spaces in `is_whitespace`, and extend `TextSprites` to accept partial line skips, for sub/superscripts or simple half line skip.

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

Merge: gamnit: cache each camera matrix and avoid boxing floats in matrix operations

Optimize the gamnit framework to avoid excessive garbage collection and a general speedup.

Replace the use of a generic `Array[E]` to implement matrices of floats, as it caused the boxing of each floats at each access. The new custom extern class does not suffer from the boxing and allow for some optimization in C code. This structure is allocated in the Nit GC (hackishly), and doesn't need an explicit liberation. A limitation of the current solution is about the size of the floating points in C, `matrix` uses doubles and `glesv2` floats, the data is copied and cast in C code, this could be optimized further as needed.

Caching each camera matrix avoids recreating them at each frame, which was quite costly, especially for the UI camera that rarely moves. The caching logic is implemented as a distinct class allowing easy debugging.

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

Merge: Nitlight: implement first_line and last_line

The existing implementation was silently droped by a previous refactorization.
Reimplement-it and add regression tests

close #2417

Pull-Request: #2429

tests: update sav//test_highlight_args1.res

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

nitc: add option --run to execute after a successful compilation.

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