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: 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>

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>

lib/java: add core services to work with native buffer in Java and JNI

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

contrib/jwrapper: add command to wrap Bitmap and nio.Buffer to the examples for reference

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

lib/c: add memmove to `CByteArray` and its native counterpart

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

hotfix: contrib/tnitter/net.xymus.tnitter.txt: terminate description

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

Merge: Tnitter: read the latest tnits on the go with the new Tnitter portable app!

Intro a portable client for Tnitter listing the more recent 16 tnits. It is more of a test and example on using `AsyncHttpRequest` to do both simple request to a REST server and implement push notifications using an open request. By design, this client does not support posting Tnits, this feature would require much more code and it may be simplified by future services in the lib.

To support the client the server has a few new REST interfaces. One to list the tnits and the other for push notifications.

This PR also fixes a few bugs on the Tnitter server: losing the session on posting and empty post being accepted by the server.

-----

Note for the reviewers: You may be better to read commits individually. There are 2 bigs commits, one extracts the database logic from the model module and the other introduces code generated by Jwrapper for the native layer of that Android ui implementation.

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

contrib/tnitter: intro .gitignore

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

tnitter: add the clean rule to Makefile

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

contrib/tnitter: add apk link to metadata

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

Merge: Add trails to nitiwiki

Wikilinks, with the directive `trail`, will register the target page as an element of a trail.
Each `trail` are chained together and will display navigational link `prev` for the previous page of the trail, `next` for the next page of the trail and `up` to go to the pages that has used the `trail` wikilink.

For instance, if the page `doc.md` has the following content:

~~~md
To use nitiwiki, first [[trail: install|install it]],
then [[trail: simple_wiki|create a first wiki]].

You can also do advanced things like:

* [[trail: github|editing pages with github]]
* [[trail: templating| adapting the templates]]
~~~

A trail will be made and will consist of the sequence of pages `install`, `simple_wiki`, `github` and `templating`.
On each one of these pages, there will be links for the previous, the next page and the `doc.md` pae.

If a page includes trail wikilinks and is also the target for trail wikilinks, then the two trails are merged and pages will be visitable in a depth-first order.
This nesting of trails can be used to have sections and sub-sections.

Demo is up: http://info.uqam.ca/~privat/nitlanguage/manual/

Pull-Request: #1830
Reviewed-by: Alexandre Terrasa <alexandre@moz-code.org>
Reviewed-by: Lucas Bajolet <r4pass@hotmail.com>

Merge: lib/core: add Text::prefix and Text::suffix

I wanted a way to detect and possibly remove a prefix from a string but there was no straight-forward way.

So I propose one:

~~~
var str = "hello world"
var p = str.prefix("hello")
assert p != null and p.text_after == " world"
assert str.prefix("fail") == null
~~~

Same with `suffix`

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

Merge: Handle gracefuly multi-varargs

This solve a very borderline issue when a signature contains more than one vararg parameter.
They are still refused in the common case but can occurs when multiple initializers are combined into a single constructor.

The implemented semantic is the following:

* vararg parameters remains varag
* if more than one vararg parameter exists in a signature, then the signature does not accepts additional arguments, it means that each vararg is associated to a single argument (a discarded alternative was to only keep the first/last parameter as the main vararg one)
* the associated argument can be either a single value, of a reverse vararg with an ellipsis.

~~~nit
class A
   fun x(i: Int...) is autoinit do end
   fun y(j: Int...) is autoinit do end
end
var a
a = new A(10, 20) # OK: i=[10], j=[20]
a = new A(10, 11, 20) # Refused
a = new A([10, 11]..., 20) # OK: i=[10, 11], j=[20]
a = new A([10, 11]..., [20, 21, 22]...) # OK: i=[10, 11], j=[20, 21, 22]
a = new A([10, 11], [20, 21, 22]) # Refused but a hint is given that `...` may be missing
~~~

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

nitiwiki: update examples with trails

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

nitiwiki: update README

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

nitiwiki: update tests

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

nitiwiki: render trail navigation on the macro-placeholder %`TRAIL%`

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

nitiwiki: add `trail:` command to collect trails

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

nitiwiki: add a simple command extraction on wikilinks.

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

contrib/tnitter: add fdroid metadata

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

lib/nitcorn: update xymus.net with latest Tnitter services

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

contrib/tnitter: refactor to set the server interface at compilation

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

contrib/tnitter: add icon

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

contrib/tnitter: intro simple app

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

contrib/tnitter: intro push notification service to the server

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

contrib/tnitter: add REST interface

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

contrib/tnitter: do not allow posting empty tnits

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

contrib/tnitter: replace `latest_posts` by a 2 args `list_posts`

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

contrib/tnitter: fix logout on post

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

contrib/tnitter: revamp how the DB is handled

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

contrib/tnitter: extract `DB` from `model` into `database`

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

lib/android: use the standard "medium" size for labels

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

lib/app/ui & platforms: intro ListLayout

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

lib/serialization: fix deserialization of Error

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

nitiwiki: add `WikiEntry::a_from` to factorize the creation of links to pages.

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

string_search: add `Text::prefix` and `Text::suffix`

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

string_search: add `Match::text_before` and `Match::text_after`

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

string_search: add nitunits to `Match`

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