dup'd binding copy all existing values into new eval scope

jruby · May 1, 2024 · 0cc945b · 0cc945b
1 parent 2b9b9d9
commit 0cc945b
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/core/src/main/java/org/jruby/runtime/DynamicScope.java b/core/src/main/java/org/jruby/runtime/DynamicScope.java
@@ -601,6 +601,11 @@ public DynamicScope cloneScope() {
     }
 
     public DynamicScope dup() {
-        return new ManyVarsDynamicScope(staticScope.duplicate(), parent);
+        DynamicScope newScope = new ManyVarsDynamicScope(staticScope.duplicate(), parent);
+        IRubyObject[] values = getValues();
+        for (int i = 0; i < values.length; i++) {
+            newScope.setValueDepthZero(values[i], i);
+        }
+        return newScope;
     }
 }
diff --git a/spec/tags/ruby/core/kernel/eval_tags.txt b/spec/tags/ruby/core/kernel/eval_tags.txt
@@ -1,3 +1,3 @@
 fails:Kernel#eval raises a LocalJumpError if there is no lambda-style closure in the chain
 fails:Kernel#eval with a magic encoding comment ignores the magic encoding comment if it is after a frozen_string_literal magic comment
-fails:Kernel#eval makes flip-flop operator work correctly
+
diff --git a/spec/tags/ruby/library/erb/result_tags.txt b/spec/tags/ruby/library/erb/result_tags.txt
diff --git a/spec/tags/ruby/library/erb/run_tags.txt b/spec/tags/ruby/library/erb/run_tags.txt