lib/android/toast: remove some callbacks

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

lib/android: intro `app::native_context` and use in `dalvik`

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

Merge: Benitlux: Minor improvements to the server

This is in preparation to the introduction of all the social network features.

I expect to have some .sav to update because of the commit removing the check of NIT_TESTING.

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

Merge: nitcc open class

Accept open classes in regular expression in nitcc.

I thought that the feature was already here, since it is in SableCC4 and already handled by the current nitcc engine.

For information, this allows to indicate an open range of accepted characters.

~~~
a = 'A'...; // `a` can match "A", "B" or "䕫" and a lot other character
~~~

It is basically equivalent to

~~~
a = Any - (#0 .. '@');
~~~

It is also used by the grammar to extends the range of possible literals and enable language description with phoenician keyword and complex emoji-based regular expressions.

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

Merge: app.nit UI: Intro an attribute to toggle password entry on TextFields

The attribute `TextField::is_password` toggles hiding the content of the field using platform specific services. So on Android, not only is the text hidden but it also notifies the soft keyboard to behave accordingly.

In the future, we could add alternative modes for email entry and more.

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

Merge: Misc text: some improvement on `join` and `to_base`

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

Merge: Interpreter FFI: custom compile_dir, use /tmp/ by default, and delete after execution

Generate native files in the `/tmp/` directory and delete them after execution. The directory can be customized using `--compile-dir`.

This should fix #1974 and #1975.

To support some other OS, we will need to use something other than `getpid` and `/tmp/`. We could also improve upon this by reusing the generated libraries between executions.

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

lib/core/text: move `to_base` to the main base module and remove the useless `signed` parameter

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

lib/core/text: add a last_separator to `Collection::join`

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

nitcc: test open class

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

nitcc: do not limit accepted literal char to `~`

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

nitcc: add open-class regexp for tokens.

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

contrib: do not check for NIT_TESTING, nitcorn already checks

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

contrib/benitlux: use result based rules to compose the server rule

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

contrib/benitlux: move the iface declaration to the toplevel for use with -D

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

contrib/benitlux: add `Beer::id`

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

lib/ios: implement `TextInput::is_password`

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

lib/android: implement `TextInput::is_password`

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

niti FFI: create compile_dir with permissions 0o700

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

lib/core: add optional mode parameter to mkdir

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

Merge: FFI: use more c_name and protect some macros for global compilation

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

niti FFI: delete the nit_compile directory after execution

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

niti FFI: cmd line option to change the nit_compile dir and default to /tmp/

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

Merge: Faster text get

Try to improve the efficiency of the FlatText[] method by shortcuting some common and easy cases.

using benchmarks/json:


Ping @R4PaSs for review

Pull-Request: #1973

Merge: lib/markdown: improve performance a bit

Using benchmarks/markdown:

* nitmd_master is before
* nitmd_md is this now
* nitmd_md_fast is with #1973 in addition
* the 3 same avain with with --semi-global
* 2 Java libs: txtmark, markdown4j


There is still a lot of performance improvements to do anyway.

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

tests: update sav//test_text_stat.res

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

lib/markdown: allow to ignore the MDlocation on Tokens

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

text/flat: FlatText::[] handle statistically common cases first

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

text/flat: FlatText::char_to_byte_index shortcut length_of_char_at if possible

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

text/flat: FlatText::char_to_byte_index do the +1 first (and 0 last)

Because stats!

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

text/flat: FlatText::char_to_byte_index factorize `item` accessor

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

text/flat: faster NativeString::set_char_at

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

lib/markdown: use a break label instead of a local flag

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

lib/markdown: avoid useless `to_s` on Chars

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

contrib/benitlux: use the latest entry as reference for the current offering

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

contrib/benitlux: accept more date formats

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

lib/app & linux: intro `TextInput::is_password`

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

Merge: fix Int::is_prime

Primality test should also test the sqrt of the number.

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

Merge: nitc: don't crash when checking if a broken method can be inlined

Pull-Request: #1968

Merge: New `optional` annotation on attributes

As requested at #1843 by @R4PaSs, a new annotation `optional` is now available on attributes.

~~~nit
class A
    var x: Int = 99 is optional
   # Because of `optional`, the automatic signature of the A constructor is `init(x: nullable Int)`
end

var a = new A
print a.x # outputs 99
var b = new A(4)
print b.x # output 4
~~~

In the model, the `optional` annotation only affects the signature and the behavior of the setter.
It transforms the argument to a `nullable` one and use the provided value if `null` is given as a parameter.

The `nullable` parameter is then propagated to the automatic constructors in the usual way.

Basically, the previous example is equivalent to having written:

~~~
class A
    var x: Int is noautoinit
    fun x=(x: nullable Int) is autoinit do if x != null then self.x = x else self.x = 99
end
~~~

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

Merge: typing: include a hook to enable more precise error information on 'expected expression' errors

The point is to help to solve the second part of #1392

Before:

~~~
print (-1).to_s
^
test.nit:2,1--10: Error: expected an expression.
~~~

After:

~~~
print (-1).to_s
^
test.nit:2,1--10: Error: expected an expression to be the receiver of `to_s`
~~~

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

fix Int::is_prime

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

nitc: don't crash when checking if a broken method can be inlined

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

src/ffi: protect friendly macro names in type check and casts for global comp

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

src/ffi: fix use of c_name for mangled names

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

tests: add base_attr_optional.nit

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

engine: handle new attribute annotation `is_optional`

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

modelize: new attribute annotation `is optional`

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

modelize: use the signature of the setter in the initializer

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

Merge: examples/calculator: fix missing icon in Android

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

typing: include a hook to enable more precise error information on 'expected expression' errors

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

examples/calculator: fix missing icon in Android

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

Merge: core/list: fix `List::clear` so it resets `length` to 0

This bug lead to segmentation faults when iterating over the items of a list after a call to `clear`.

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

Merge: contrib/opportunity: fix usage of % format, now indexed from 0

http://xymus.net/opportunity/ has already been updated with the fix.

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

Merge: lib/sdl: workaround to avoid constructor callback bug

This was first a temporary fix for #1941 to repair Nit games on the deskop. But it is also a nice cleanup, it brings to Nit the instantiation and the implicit cast to a more general and nullable type. The casts themselves were not very pretty using FFI callbacks.

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

Merge: Opportunity and Tnitter logos

This PR intro 2 logos for Opportunity (in English and French) and a new wide icon for Tnitter. These news icons are already in use on http://xymus.net/opportunity.

Note that even if the large Opportunity logo on the landing page is vectorial, it should not be the file included in the `art/` directory for a production server. That file has both logos, more assets and uses fonts, all of which should be stripped for a production server. It is still good enough for local debugging.

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

Merge: Minor fix to atan2 and triangulate

Fix reversed axes in `atan2`, but since it is actually just a change in the name of the variables (and doc) it should not break the clients.

Also sorting points of a polygon in `triangulate` completely invalidated some complex polygons.

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

Merge: Minor tweaks and fixes for Android

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

lib/core: fix `List::clear` not setting `length` to 0

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

contrib/opportunity: fix usage of % format indexed at 0

Signed-off-by: Alexis Laferrière <alexis.laf@xymus.net>
Reported-by: Lucas Bajolet <r4pass@hotmail.com>

xymus.net: point to subfolders instead of subdomains to share the cache

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

xymus.net: track more pages with google analytics

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

xymus.net: use opportunity and tnitter branding in header

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

contrib/tnitter: intro wide icon for the web client

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

contrib/opportunity: revamp style of welcome page and use branding

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

contrib/opportunity: intro branding source file

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

lib/core: fix order of the params of `atan(y, x)`

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

lib/geometry: do not sort points as it invalidates the shape of the polygon

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

Merge: Recover `recover_with`

close #1767

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

Merge: more work on nitcc

Random nitpicks

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

test_regex_check.nit: use the shorter `join`

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

lib/core/collection: improve the default `has_all` for basic cases

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

nitcc: improve perf by caching the result of cmangle

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

nitcc: fixup the .lr.out file

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

lib/sdl: fix for constructor callback bug

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

contrib: use add_all instead of recover_with

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

nitc: do not define be32toh when it exists on Android

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

lib/android: use the more general NativeContext with annotations

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

lib/android: pin Nit buttons from Java callbacks

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

lib/android: fix ListLayout by pinning the adapter in the Java GC

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

code: update clients of the former `recover_with`

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

lib/core: rename `recover_with` as `add_all`

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

Merge: OSX: disable non-working tests, and tests cocoa

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

tests: add ncurses and  x11 to OSX toto

So tests pass with *todo* (instead of *fai*l)

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

tests: enable testing cococa on OSX

What was to point to disable cocoa on OSX anyway?

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

Merge: Use a subfolder `android` for Android related files like `res` and `libs`

Move Android specific files of each projects to a subfolder `android`. This includes the `res` folder with custom resources and icons, as well as the `libs` folder storing Android libraries (jar files). It is a followup to the iOS support where platform specific icons are stored in the subfolder `ios`.

Also remove the quiet option (`-q`) from the call to ant, because it often hid error messages. Besides, the call to ndk-build is already very verbose, so a longer output from ant is not that bad. In the future, for a cleaner output, we could try to replace ant with gradle.

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

Merge: nitcc fix issue when a DFA node has only a single Any transition

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

nitcc: add test/lexer-any.sablecc

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

nitcc: Handle case of DFA state with a unique Any transition

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

contrib & examples: update Android apps

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

nitc android: do not use quiet the call to ant as it hides some errors

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

nitc android: move android specific folders (res/libs) to an android folder

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

Merge: Revamp `Text::format`

Change the behavior of `Text::format` so it uses an index from 0 (instead of 1), and the escape sequence to a double percentage sign (ex, `%%4` to get `%4`). Also, now `format` ignores `%` that are not followed by a number.

There were problems with the previous escape sequence (using `\%`):
1. In literal strings, the `\` itself has to be escaped, so a whole escaped sequence look like `\\%4`.
2. The extra `\\` were not removed from the resulting string, so it was still impossible to produce a `%4` because `\\%4` resulted in `\\%4`.

These changes were prompted by an error in the code generated the i18n phase on a lone `%` not related to the `format` method. Also, the index beginning at 1 were always confusing to me, it was a different behavior than `Array` and other

@R4PaSs I would like your opinion on this.

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

Merge: nitcorn: cache served files, custom JS for FileServer and do not depend on curl

This PR extends `FileServer` with caching of served files and custom JavaScript code for the header. The caching is set to 1h by default, but it can be customized.

Removing the default import of the `proxy` module with nitcorn, it depends on curl, and thus adds a (small) overhead to all nitcorn programs and need the library. This could be reverted when `proxy` is updated to use only libevent.

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

Merge: iOS icon support and fix black bars on physical devices

This PR adds support for custom icons on iOS by adding an `Assets.xcassets` folder to the generated Xcode project. It is used to store a user provided icon specified in the `ios/AppIcon.appiconset` folder from the root of the Nit project. This is similar to the Android support where the icon files are stored in the `res` folder.

Also adds `LaunchScreen.storyboard` to the generated project. This file defines a kind of splash screen displayed while the app loads. The main advantage of this file is that it fixes a bug on physical devices where black bars were visible at the top and bottom of the screen.

The launch screen shows the app name in the center with _app.nit_ at the bottom. This is the default format provided by Xcode but it does not follow the official guidelines. In the future we may want an option for the user to provide a custom file.

To generete these icons, this PR update `svg_to_icons` with support for the icon formats expected by Xcode.

As examples and test, both Tnitter and calculator are updated with icons for iOS.

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

tests: niti and nitvm don't support iOS

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

tests: update iOS tests with files generated from the LaunchScreen

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

nitc: add -Wno-trigraph to CFLAGS

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

contrib/tnitter: add icon for iOS

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

contrib/tnitter: convert font to path for portability

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