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

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>

examples/calculator: intro icon for iOS

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

contrib/inkscape_tools: update README

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

contrib/inkscape_tools: create target folder if it does not exists

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

contrib/inkscape_tools: add iOS target format

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

lib/ios: document how to set an app icon

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

lib/ios: fix unicode whitespace, was reported as a warning by clang

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

frontend/i18n: escape % in internationalized strings

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

lib/text: `Text::format` harder to break and using `%%` as escape

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

lib/text: `Text::format` uses an index starting at 0, as `Array` does

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

lib/text: `FlatString::substring` returns "" on count == 0 or less

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

nitc/ios: add Assets.xcassets

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

lib/nitcorn: cache files served by `FileServer` for 1 hour

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

lib/nitcorn: intro the attribute `FileServer::javascript_header`

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

lib/nitcorn: do not include `proxy` with nitcorn by default (it requires curl)

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

nitc/ios: add LoadingScreen.storyboard

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

nitc/ios: remove superfluous content of the generated PBX project file

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

Merge: iOS: implement data_store and missing life cycle callbacks

Implement the main missing features of _app.nit_ on iOS: `data_store` and life-cycle hooks (like `on_save_state`). These changes can be seen on the calculator app, as it preserves its context using the `data_store`.

`data_store` is implemented with `NSUserDefaults` to store objects sertialized to Json. It is very similar to Android's implementation using shared preferences. This may be a bit limited as it is not meant to hold large strings, and some data objects (like game saves) should instead be saved to a file.

This PR also implements all life-cycle callbacks in iOS, until now only `on_create` was implemented. We may have to update _app.nit_ life-cycle to fit better with the life-cycle of iOS, the states between a running app and a fully stopped app are different between iOS and Android. I'm thinking of removing the two callbacks on_start/on_stop and keep only the more general callbacks on_create/on_destroy and on_resume/on_pause, and the services on_restore_state/on_save_state.

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

lib/serialization: serialize all insances of Text as we did with srings

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

lib/ios: execute missing callbacks of app.nit on iOS

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

lib/ios: implement data_store using NSUserDefaults

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

lib/ios: wrap NSUserDefaults

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

Merge: Small cleanups related to constructors

Nothing fancy. Just small things extracted from a massive wip branch

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

Merge: Android support: better doc and libgc compilation service support latest NDK

Update the doc of the Android package with better instructions on how to configure the host system before compiling Nit applications for Android. Also fix the libgc build script to support the latest NDKs.

@privat The change to the script may require to install the latest NDK on the test server. Current users of the Android compilation should not have to do anything, only new installations should require to also install the latest NDK.

Pull-Request: #1944

Merge: Intro support for async HTTP requests and ListLayout for iOS

This PR completes the implementation app::UI for iOS by implementing `ListLayout` using a `UITableView`. It also introduces the implementation of `http_request` using services from the foundation framework.

These features enable the Tnitter portable application to be compiled for iOS without modification.

The remaining API of app.nit still missing on IOS are `data_store`, `audio` and maybe `assets`.  At this point, only `data_store` is required for fully featured portable applications using the native UI.

---

As with #1942, some generated code has been commented so it can be reactivated when useful and when the needed types are also wrapped.

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

rta: improve code for methods of classes

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

niti: rename `args` as `arguments` in `call`

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

nitc: remove the redundant property `mfree_init`

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

sdl: remove old style init

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

dino: remove old style init

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

contrib/tnitter: add iOS app rule

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

lib/ios: implement `app::http_request`

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

lib/ios: complete implementation of app::ui with support for ListLayout

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

lib/cocoa: intro wrappers for NSData, NSError and NSIndexPath

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

lib/cocoa: intro `NSString::nil

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

nitc: add permission for HTTP requests to iOS apps

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

lib/pthreads: do not include libgc on iOS

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

share/libgc: update tools_dir to support the latest Android NDK

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