Review request for JDK-8006529
Jim Laskey (Oracle)
james.laskey at oracle.com
Wed Jan 30 08:20:37 PST 2013
FinalizeTypes.java
549 private static void updateSymbols(final Block block) {
550 if (block instanceof FunctionNode && block.getFunction() == block) {
Why not a visitor (instead of instanceof test)?
and Shouldn't it be?
549 private static void updateSymbols(final Block block) {
550 if (block instanceof FunctionNode) {
551 assert block.getFunction() == block;
I have to say jdk.nashorn.internal.codegen.CompilerAccess creeps me out a bit. Since it's for testing only, it should not get into fcs build.
+1 otherwise.
On 2013-01-30, at 11:53 AM, Attila Szegedi <attila.szegedi at oracle.com> wrote:
> Please review JDK-8006529 at http://cr.openjdk.java.net/~attila/8006529/webrev.00
>
> Thanks,
> Attila.
More information about the nashorn-dev
mailing list