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


Trivial merge