Correctly set isModuleBody for parent frames for #eval

* This fixes `eval "1.times { yield }"` to raise a SyntaxError like CRuby (and not a LocalJumpError like before).
oracle · Jun 21, 2022 · 3e4b726 · 3e4b726
1 parent e8a903b
commit 3e4b726
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/main/java/org/truffleruby/parser/TranslatorDriver.java b/src/main/java/org/truffleruby/parser/TranslatorDriver.java
@@ -449,13 +449,15 @@ private TranslatorEnvironment environmentForFrame(RubyContext context, Materiali
                     null);
             final MaterializedFrame parent = RubyArguments.getDeclarationFrame(frame);
             assert (blockDepth == 0) == (parent == null);
+            boolean isModuleBody = blockDepth == 0 &&
+                    RubyArguments.getMethod(frame).getSharedMethodInfo().isModuleBody();
 
             return new TranslatorEnvironment(
                     environmentForFrame(context, parent, blockDepth - 1),
                     parseEnvironment,
                     parseEnvironment.allocateReturnID(),
                     true,
-                    false,
+                    isModuleBody,
                     sharedMethodInfo,
                     sharedMethodInfo.getMethodName(),
                     blockDepth,