Review request for JDK-8067774

Marcus Lagergren marcus.lagergren at
Wed Dec 17 17:03:23 UTC 2014

Well this was significantly smaller than I thought and also a simplication from the previous implementation. Well done for a bugfix!



> On 17 Dec 2014, at 17:45, Attila Szegedi <attila.szegedi at> wrote:
> Please review JDK-8067774 at <> for <>
> This fixes the issue reported with asciidoctor. The change is not trivial – I had to introduce a change in the way LocalVariableTypeCalculator performs its expression type calculations; it now uses a type stack. (If I want to be honest, I should have realized this is the right way to implement it from the beginning… Oh well, live and learn.) As a beneficial consequence, a lot of code actually became simpler; the whole business with using a Symbol->Type function to evaluate expression types is no longer necessary (all changes in the various expression classes are to do with removal of that logic).
> Thanks,
>  Attila.

More information about the nashorn-dev mailing list