tests: add bench_seq

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

core/array: provide a better implementation of `copy_to` (not native yet)

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

core/array: fix implementation of naive copy_to to avoid overwriting

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

core/array: use `copy_to` to factorize some code

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

Merge: Modelize check visibility

Forces modelize to check that the return types of methods are legal with regard to visibility.

Also fixes some code where such issues occur.

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

android: make native audio_manager access private

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

tests: update sav because changes in visibility

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

tests: improve base_prot_sig

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

modelize: always check visibility of return types

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

src/rta: import csv publicly.

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

nitc/ffi: make private some top-level methods

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

core/text: Text::substrings is private

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

core: remove useless return types

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

Merge: mongodb: allow client to set `skip` and `limit` when calling `find`

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

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

Merge: github: save Github events ids.

Save event IDs so the future loader will know it will not have to process it again.

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

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

Merge: mongo: fixes more crashes from mongodb api

This PR add two more fixes since #1855 :
* use `FinalizableOnce` instead of `Finalizable`
* copy bson structs returned by `mongoc_cursor_item`

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

Merge: Benitlux: fix separator between beers name and description

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

Merge: console: add simple progress bar that refreshes itself

Example:

~~~nit
import console

var pb = new TermProgress(10, 0)

for i in [1..10] do
pb.update(i)
end
~~~

Will display something like:
~~~sh
20% [==========================>                                       ]
~~~

See documentation for more info about the progress bar.

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

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

Merge: Add error instead of segfault on gethostbyname fail

Signed-off-by: Philippe Pepos Petitclerc <ppeposp@gmail.com>

Pull-Request: #1857
Reviewed-by: Jean-Philippe Caissy <jpcaissy@piji.ca>
Reviewed-by: Jean Privat <jean@pryen.org>

mongodb: fix iterator crashes

Before this PR, the iterator `is_ok` status relied on `mongoc_cursor_more` that can return
true even if the iterator is terminated.
This PR fixes the problem by using the result of `mongoc_cursor_next` instead.

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

mongodb: allow client to set `skip` and `limit` when calling `find`

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

mongodb: fixes crash when the return of `mongoc_cursor_current` was freed

As explained by the documentation (that I should have read more thoroughly...),
`bson_t` structures allocated by the `mongoc_cursor_current` function should not be freed by the user.

Before this commit, the `bson_t` returned by `mongoc_cursor_current` where left as this then freed
by the nit GC.

This commit makes the `current` method to return a copy of the `bson_t` struct so
it can be freed safely by the GC later.

And... this is why I fear and hate the C programming language... :)

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

github: save Github events ids.

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

mongodb: use FinalizableOnce instead of Finalizable

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

console: add simple progress bar that refresh itself

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

Merge: github/api: avoid crashes with unparsable responses

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

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

Merge: mongodb: pin non-native instances to avoid GC crashes

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

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

Merge: nitiwiki: fix multiple trails

Fix a bug where independent trails could have been merged into a big one.

Pull-Request: #1854
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>

Add error instead of segfault on gethostbyname fail

Signed-off-by: Philippe Pepos Petitclerc <ppeposp@gmail.com>

github/api: avoid crashes with unparsable responses

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

mongodb: pin non-native instances to avoid GC crashes

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

nitiwiki: extends example with a second independent trail

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

nitiwiki: fix a bug where multiple independent trails could have been merged

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

Merge: nitrpg: misc fixes

Many fixes for NitRPG in prepatation for the next PR wich introduces the use of MongoDB.

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

nitrpg: fix PlayerXCommits achievement trigger

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

nitrpg: events and achievements have owners

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

nitrpg: better organization for stats objects

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

nitrpg: use names as keys

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

Merge: Simple threadpool implementation

A really basic implementation of a threadpool, using an array as queue for now.

Pull-Request: #1848
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Jean Privat <jean@pryen.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Skipping threadpool_example

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

simple threadpool implementation

Signed-off-by: BlackMinou <romain.chanoir@viacesi.fr>

Merge: Bytes update

Various updates to `Bytes`, added a new escaping method taking into account \u and \x escape sequences

Also added a add_char method to `Bytes` for convenience.

`to_bytes` in Text now has an optional length parameter for those who wish not to rely on `cstring_length` (or did not want to pay the cost) for the length in bytes.

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

contrib/benitlux_daily: use only "- " to separate the name from the description

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

contrib/benitlux_daily: add verbose option for debug

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

lib/core: Added escaping method to Bytes for use in compiler

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

Merge: Prefix grammar

Update of the grammar to support the enhancement of literal strings and chars via prefixes (see issue #1734)

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

lib/core: Added copy method `to_bytes_with_copy` in Text

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

lib/core: Added `add_char` method to `Bytes`

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

lib/markdown: Fixed spacing issue in which a `then` was mistaken for a suffix

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

parser: Regenerated tables for previous commit

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

parser: Updated grammar to support prefixed and suffixed `String` and `Char`

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

Merge: nitrpg: update the userscript to the new github layout and fix todos

Basically a rewrite, but it works on my computer

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

Merge: svg_to_icons: add an option to generate icons with different names

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

Merge: Improve the doc of `CurlHTTPRequest`

This is still far from an ideal doc but its better than nothing.

This doc will be useful for #1845.

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

lib/neo4j: fix conflict on `data` in custom implementations of post requests

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

nitrpg: update the userscript to the new github layout and fix todos

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

Merge: model: fix `try_get_primitive_method`

The implementation was an old code that did not use modern facilities,
moreover it was broken anyway.

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

Pull-Request: #1844
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>

lib/curl: better doc for `CurlHTTPRequest`

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

lib/curl: `CurlHTTPRequest::datas` to `data`

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

model: fix `try_get_primitive_method`

The implementation was an old code that did not use modern facilities,
moreover it was broken anyway.

Reported-by: Lucas Bajolet <r4pass@hotmail.com>
Signed-off-by: Jean Privat <jean@pryen.org>

contrib/inkscape_tools: add option to generate icons with different names

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

Merge: Tests: cleanup old_style_init in some tests

In preparation of a future PR on constructors (cf. #1800), this is a simple cleanup (and bugfix) of some tests.

Pull-Request: #1842
Reviewed-by: Alexis Laferrière <alexis.laf@xymus.net>
Reviewed-by: Romain Chanoir <romain.chanoir@viacesi.fr>

tests: update init in (de)serialization

also fix bug where `n` was not correctly set (new constructors powa!)

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

tests: remove some old_style_init

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

tests: remove deprecated test base_attr_def

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

Merge: Jwrapper: Fix bug with nested classes

Fixes the nested classes bug in jwrapper.

Splits JNI full name and java full name.
Generated comments use JNI full name.

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

Rename JavaClass package_name to full_java_name

Signed-off-by: Frédéric Vachon <fredvac@gmail.com>

Use JNI full name in generated comments

Signed-off-by: Frédéric Vachon <fredvac@gmail.com>

Merge: Performance fix for comparison in FlatString

As reported in #1838, a lot of unnecessary unboxings were done when comparing bytes in FlatString, this should be fixed by short-cutting the access to the bytes of a FlatString via `_items`.

Also while I was at it, the operation is now optimized when working on any `FlatText`

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

tests: Update sav for stats test

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

lib/core: Allow comparison operators to work on any subclass of FlatText

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

lib/core: Perfize < for FlatString which caused a lot of unneccessary boxings

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

Splits JNI and Java full name

Signed-off-by: Frédéric Vachon <fredvac@gmail.com>

Merge: android: use the Android Java API to load pixel data instead of libpng

Use the Android Java API to load and decompress image files, and remove the old use of a local copy of libpng. This new system is much more simple, produces smaller APK files (~30% smaller) and supports more image formats, including jpeg. It relies on the Java FFI to access the API and the C FFI to move pixels around.

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

lib/c: rename memmove to move

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

Merge: Clean up constructors in core/text

Remove useless named init when automatic-constructor are sufficient, and remove old_style_init when named init are sufficient.

This lib cleanup is part of an ongoing cleanup of constructors, see #1800

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

Merge: tests: add test_sort_perf.nit

This test illustrate a current issue with string comparisons.
On my machine, a valgrind of it gives 80+% on BOX_Byte.

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

Merge: libevent & nitcorn: close connections on error and implement callbacks in Nit

This PR moves the implementation of libevent callbacks from C to Nit to enable specialization and refinements of the callbacks, it should help for #1803. Also the event callback closes the connection on the Nit side on errors so servers do not attempt to write on broken connections, it should fix the current crash on http://xymus.net/ with Tnitter clients.

Also update the only user of `read_callback_native` for the new API. And seize this opportunity to rename the module, replace class refinement with specialization, clean up the doc and optimize a bit how it handle strings.

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

core/ropes: remove old_style_init, use only named init (not ideal but better)

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

core/flat: remove named init `with_pos` and use auto-constructor

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

lib/mnit: pad loaded images to the closest power of 2 on Android

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

lib/mnit: Android images always have an alpha channel once decoded

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

lib/android: revert Android's multiplication of color values with alpha

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

tests: add test_sort_perf.nit

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

lib/nitcorn: http_request_buffer feed the request to read_http_request

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

lib/nitcorn: update doc and style of http_request_buffer

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

lib/nitcorn: http_request_buffer subclass Connection instead of refining it

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

lib/nitcorn: slightly optimize http_request_buffer

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

lib/nitcorn: rename http_request_parser to http_request_buffer

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

lib/libevent: move error callback to Nit and close connection on error

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

lib/libevent: move read and write callbacks implementation from C to Nit

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

lib/libevent: wrap more services from the C API to access from Nit

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

lib/libevent: remove duplicated check on `closed`

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

lib/libevent: do not write on closed connections

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

share: remove the local copy of libpng

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

src/android: do not link with libpng

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

lib/mnit: replace the use of libpng by load_image

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

lib/android: intro a service to unpack pixel data from image files

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

lib/android: add more services to NativeBitmap

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

lib/android: `AssetManager::bitmap` only has to return a local reference

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