Adding code smell detection :

Long class
Long method parameter list
Long size method
Feature envy
Adding a visitor to analyse the contents of the methods

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

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>

matrix: use a custom low-level C structure to avoid boxing of param types

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

nitweb: add a low-level `api/light` with highlight

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

popcorn/pop_handlers: HttpResponse::json_* accepts any Serializable

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

nitlight_aas: offer a low-level JSON service

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

tests: add more tests for nitlight

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

nitlight: implements --first-line and --last-line

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

gamnit: intro cache for `Camera::mvp_matrix`

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

tests: update error_class_glob

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

gamnit: support partial line forward and backward

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

core: add non-breaking space to `is_whitespace`

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

Merge: Update legal stuff

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

Pull-Request: #2416

Merge: git-gen-version: Simplify a `if`

Extracted from PR #2201.

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

Pull-Request: #2418

Merge: makefiles: Remove dead code

Extracted from PR #2201.

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

Pull-Request: #2419

Merge: gamnit: make the template project friendlier for new users

The `template` project (previously `template_flat`) is a basic project stub that can be built upon to create a gamnit game. This PR should make it a bit easier to copy and rename the project, ensure compatibility with the nit:latest-full Docker, rely on a visible `mkdir` instead of a hidden `.gitignore`, provide a positive first `make` and suggest to implement unit tests.

Pull-Request: #2421

Merge: highlight: give more control to clients and fix regressions

Allow clients to send HTML to `new_dropdown`. Fix two regression bugs introduced by #2231: support for `CallSite` and `MVirtualType`. And as usual, also fix a few local warnings and typos.

This is used by my landing page generator script.

Pull-Request: #2423

Merge: Makefile: Don’t invoke `src/Makefile` twice

Extracted from PR #2201.

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

Pull-Request: #2424

app.nit: add default behavior to `TextAsset::load` for unit tests

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

app.nit: exit and print an error if not bound to a platform

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

Makefile: Don’t invoke `src/Makefile` twice

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

app.nit: id the platform at runtime

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

gamnit: fix nitunit

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

Makefile: Document `full` more accurately

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

highlight: new_dropdown accepts raw html

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

src: misc typos, doc and warning fixes

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

highlight: fix broken support for `MVirtualType`

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

model: bring back call site doc to highlight as `CallSite::mdoc_or_fallback`

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

gamnit: shorten template_flat to template and update namespace

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

gamnit: add the check rule to the template makefile and remove all

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

gamnit & contrib: lower the Android target API

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

tinks/Makefile: Remove an undefined phony target

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

tests/Makefile: Remove unused macro

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

Makefile: Remove unused macro

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

git-gen-version: Simplify a `if`

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

gamnit: template use `mkdir bin` instead of a .gitignore for visibility

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

NOTICE: Update

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

license: add GNU GPLv3

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

Merge: gamnit: fix gamepad path, pixelated minification, text X align and low-opacity textures

A few fixes to gamnit:

* Fix path of virtual gamepad texture.
* Fix the look of `pixelated` textures when minified (zoomed out).
* Fix X align of `TextSprites` so the first letter starts from the given coordinate.
* Lower the threshold to discard transparent fragments.

Pull-Request: #2415

Merge: opportunity: official docker image

Now that nitcorn can listen on any interface, why not having lightweight standalone web applications?

How to build?
1. install docker
2. cd nit/contrib/opportunity/docker
3. docker build --tag opportunity .

How to run?
1. docker run -d -p 8080:8080 --name  opportunity opportunity
2. xdg-open http://localhost:8080
3. docker stop opportunity

or just get it from the [hub](https://hub.docker.com/r/nitlang/opportunity/tags/)
1. docker run -d -p 8080:8080 --name opportunity nitlang/opportunity

Pull-Request: #2284
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Jean-Philippe Caissy <jpcaissy@piji.ca>

gamnit: fix virtual gamepad asset name

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

gamnit: lower transparency threshold to discard fragments

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

gamnit: fix X position of first char of a TextSprites

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

gamnit: fix pixelated changing only magnification behavior

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

Merge: Check binary files that might be wrongly commited

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

opportunity: official docker image

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

opportunity: optional argument is the iface

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

popcorn/README: remove a unexpected control character ^S

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

remove wrongly committed binary file `lib/core/collection/range`

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

jenkins: add checkbinaryfiles.sh

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

nitlight: move examples/nitlight_as_a_service stuff to highlight.nit

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

Merge: Nitin catches the runtime errors

~~~
-->abort
1,1--5: Runtime error: Aborted
abort
^
,---- Stack trace -- - -  -
| input-1$Sys$main (1,1--5)
`------------------- - -  -
-->fun toto is abstract
-->toto
1,1--20: Runtime error: Abstract method `toto` called on `Sys`
fun toto is abstract
^
,---- Stack trace -- - -  -
| input-2$Sys$toto (1,1--20)
| input-3$Sys$main (1,1--4)
`------------------- - -  -
-->class A
...var a: String is noautoinit
...fun foo do print a
...end
-->(new A).foo
3,18: Runtime error: Uninitialized attribute _a
fun foo do print a
                 ^
,---- Stack trace -- - -  -
| input-4$A$foo (3,18)
| input-5$Sys$main (1,1--11)
`------------------- - -  -
-->
~~~

Pull-Request: #2413

nitin/README: runtime errors/aborts return to the interactive loop

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

tests: add some runtime error in nitin.input

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

nitin: use the secret `catch` block to intercept runtime errors

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

Merge: Catch em all

Extends the `do catch` construct to catch all run time errors.
The interpreter also keep the error information if one wants it.

Pull-Request: #2411

Merge: Nitin tests

Add a basic test for the nit interactive interpreter.

I also added a new engine but most test fails because:

Local variables in main are too much local, so the following fail because `a` is unknown in the second line. I plan to solve this problem.

~~~
var a = 1
print a
~~~

Classes and top-level methods are analysed one after the other so the following fails because `B` is unknown while processing the first class. I do not plan to solve this problem as I think this is the expected behavior when interactive.

~~~
class A
   var b: B
end
class B
end
~~~

Pull-Request: #2410

misc: update highlighters and indenters

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

tests: add test_catch_runtime

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

nitc: add `add_raw_throw` so raw_abortions are caught

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

nitc: document add_raw_abort

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

niti: make `catch_count` writable

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

niti: keep the fatal_error information

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

niti: all fatal errors are catchable, not just aborts

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

tests: to add a nitin engine

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

nitin: add option --no-prompt to ease testing

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

tests: add tests for nitin

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

Merge: nitin, an interactive interpreter

# An experimental Nit interactive interpreter

This tool is outside src/ because:

1. Is is greatly experimental
2. It can depend on readline (GPL3) whereas the rest of nitc is Apache2.
   Both are compatible but the final binary result is GLP3.

## Features

* use GNU readline to read lines
* use importation/refinement to handle incremental execution (so basically everything works out of the box)
* maintain an interpreter and live objects (the model grows but the interpreter and runtime data are reused)

Main missing features

* top-level variables are local
* runtime error/aborts just abort the interactive loop
* FFI is strange
* No model/object inspection

## Examples

`-->` is the prompt, `...` the continuation prompt. What follows is the user input.
The rest is the output.

### Basic instructions

~~~raw
$ nitin
--> print 5+2
7
~~~

### Complex and control statements

~~~raw
-->for i in [0..5[ do
...print i
...end
0
1
2
3
4
~~~

You can use `do` blocks to delay the execution and control the scope of variables.

~~~raw
-->do
...var sum=0
...for i in [0..50[ do
...sum += i
...end
...print sum
...end
1225
~~~

### Classes and methods

~~~raw
-->class A
...fun foo do
...print "hello"
...end
...end
-->(new A).foo
hello
~~~

### Error management

In case of static errors, the history (up arrow) can be reused and updated.

~~~raw
-->class
...end
end
^: Syntax Error: unexpected keyword 'end'.
-->class A   **up arrow and update, thanks readline**
end
1,7: Redef Error: `A` is an imported class. Add the `redef` keyword to refine it.
class A
      ^
-->redef class A **up arrow and update again, thanks readline**
redef fun foo do print "Bye"
end
-->(new A).foo
bye
~~~

### Class refinement

Already instantiated objects gain the new methods, attributes and specializations.
However, the new attributes are left uninitialized (default values or init are not recomputed on existing objects)

Top-level methods automatically refine Sys.

~~~raw
-->foo
1,1--3: Error: method or variable `foo` unknown in `Sys`.
foo
^
-->fun foo do
...print "I'm sys"
...end
-->foo
I'm sys
~~~

You can store global variables as attributes of Sys

~~~raw
-->redef class Sys
...var my_int: Int is writable
...end
-->my_int = 5
-->print my_int
5
~~~

### Dynamic importation

~~~
-->print([0..10[.to_a.to_json)
1,20--26: Error: method `to_json` does not exists in `Array[Int]`.
print([0..10[.to_a.to_json)
                   ^
-->import json
-->print([0..10[.to_a.to_json)
[0,1,2,3,4,5,6,7,8,9]
~~~

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

contrib/nitin: new interactive interpreter proof of concept

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

nitc/toolcontext: make error_count writable

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

nitc/loader: always build the module importation

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

nitc/loader: load_rt_module returns the MModule

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

nitc/interpreter: NaiveInterpreter::mainmodule is writable

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

Merge: gamnit: virtual gamepad

Intro an easy to use virtual gamepad for quick ports of desktop games to mobiles devices with a touchscreen only. The gamepad is composed of up to 2 d-pads and up to 6 action buttons. Each button and direction on the pad is mapped to customizable keyboard keys. As such, it is ideal to simulate keyboard inputs on mobile devices.

The `virtual_gamepad` module uses the `app_files` annotation which automatically includes the required PNG files in Android's APK files when the module is imported.

The virtual gamepad is derived from the one in Asteronits, which it replaced. It is now used in Asteronits and Action Nitro, and it will be added to Devil's Avocados as soon as this is merged. The gamepad comes with a selection of button allowing for some customization, however custom buttons are also supported.

Usage example from Devil's Avocados:
~~~
var gamepad = new VirtualGamepad
gamepad.add_dpad
gamepad.add_button("q", gamepad_spritesheet.turn_left)
gamepad.add_button("e", gamepad_spritesheet.turn_right)
gamepad.add_button("space", gamepad_spritesheet.star)
gamepad.visible = true
app.gamepad = gamepad
~~~

And the result, on Android:


This version should manage quite well the edge cases of touchscreen controls, which explains the complexity of `virtual_gamepad.nit`. However it is still limited, future work could add:
- Virtual analog joystick, it would be easier to code than the d-pad.
- Abstraction for gamepads (and analog joysticks). This would require unification with the current Android physical gamepad support and implementing desktop physical gamepad support with SDL2.

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

Merge: gamnit: abstract camera scroll and zoom service

The new service `accept_scroll_and_zoom` lets the user adjust the camera using platform specific controls. On desktop computers, the mouse wheel controls the zoom and the middle mouse button drags the world/camera. On Android, a pinch gesture controls the zoom and 2 fingers slide the scroll.

This reuse the previous `accept_two_fingers_motion` and intro an implementation for desktop computers. It will be used by, at least, Sputnit in which it will replace the current implementation which is broken since the upgrade to SDL2.

Pull-Request: #2407

Merge: gamnit: intro dynamic resolution support

The dynamic resolution can change while the game is running to reduce the load on the GPU when more performance is needed. So an action scene may display tons of explosions and maintain a good framerate by reducing the number of pixels on screen, and a calm scene may gain back the full resolution to better show details.

At this point, the dynamic resolution must be set manually by the user (using something like a video quality slider) or by some heuristic. Future improvement could autodetect skipped frame and set the resolution accordingly.

While the current set of gamnit games rarely overload a GPU, there is many scenarios where a dynamic resolution can be useful: mobile device with a high-resolution display and a limited battery, 4K Android TVs and a VR headset that must maintain 90 FPS.

In Asteronits, the keys `, and .` change the dynamic resolution. Here are a few screenshots of the results.

At 1.0, the native resolution:

At 0.25:

At 0.03: (I wouldn't recommend using this value by default)

Pull-Request: #2408

gamnit: world_camera position defaults to 1080 pixels in height

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

asteronits: change dynamic resolution with , and .

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

gamnit: intro dynamic resolution support

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

glesv2: intro param to get names of bound entities

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

glesv2: small (but important) fixes

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

gamnit: implement `camera_control` for linux

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

gamnit: intro `camera_control` an abstraction of `accept_two_fingers_motion`

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

sdl2 & gamnit: add mouse wheel events

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

action_nitro: add f-droid package description file

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

gamnit: fix clearing reference to context in sprites too

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

gamnit: add example to `gamnit::keys`

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

Merge: inkscape_tools: support Inkscape versions before and after 0.92

Inkscape changed the default DPI at version 0.92. This PR improves `svg_to_png_and_nit` to set the default DPI according to the detected Inkscape version, 90 DPI up until version 0.92 and 96 DPI for version 0.92 and later.

Reference: http://wiki.inkscape.org/wiki/index.php/Release_notes/0.92#Important_changes

This is a more portable solution than #2374.

Pull-Request: #2406

Merge: GC now collect cycles

This fixes an issue with the actor model where the GC is unable to collect cycles associated with actors objects and printing an annoying warning

Theses modifs should fix the tests in #2401

The downside is that the finalization is no longer in topological order

Pull-Request: #2404

Merge: Some minor fixes to actors examples

Tried to follow @tremblay-guy commentaries on #2361  to improve the code of the actors examples

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

action_nitro: intro Android client with gamepad and icon

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

action_nitro: general gameplay tweaks

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

asteronits: use the new touch gamepad

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

gamnit: intro virtual gamepad

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

inkscape_tools: support Inkscape versions before and after 0.92

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