Merge: Misc AST
authorJean Privat <jean@pryen.org>
Thu, 9 Apr 2015 05:24:43 +0000 (12:24 +0700)
committerJean Privat <jean@pryen.org>
Thu, 9 Apr 2015 05:24:43 +0000 (12:24 +0700)
commit34a39fdab13dc009cf40ec1a3ef5f661b743acb8
tree9219df6d14892e3cdb001fe55d199def72d780f0
parent90df56f73c160350d61a2a641271c6dca5904fb0
parentc183e2a9325f2316f82785be1e4806befbb4e93d
Merge: Misc AST

Various small improvements on the ast side.

The main improvement is the lower consumption of Location instances.
Before: 681k where created; for 174MIr
After:  472k are created (-30%); for 120MIr (-31%)
However, the final impact should be negligible.

Other changes are related to error messages in order to have them more useful in other context.
As a proof of concept, nitlight can show some warnings and errors in the generated HTML.
There is still issue, especially: how to display them property. Currently they are in a popupable dropdown boxes, but the with is too small and nested info boxes show unreadable stacked  things.

Eg, for `return new Foo[nullable Bar]`, how can I show

* information on the classes Foo and Bar
* error on the whole `nullable Bar` because Foo is in fact not generic
* error on the whole `new Foo[nullable Bar]` because in fact we are in a procedure, so no return value is expected

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