[Truffle] More explicit name since the logic only makes sense for cla…

…ss variables.
jruby · Feb 24, 2016 · cf89d2f · cf89d2f
1 parent 078e459
commit cf89d2f
Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/truffle/src/main/java/org/jruby/truffle/language/LexicalScope.java b/truffle/src/main/java/org/jruby/truffle/language/LexicalScope.java
@@ -44,7 +44,7 @@ public void unsafeSetLiveModule(DynamicObject liveModule) {
     }
 
     @TruffleBoundary
-    public DynamicObject resolveTargetModule() {
+    public DynamicObject resolveTargetModuleForClassVariables() {
         LexicalScope scope = this;
 
         // MRI logic: ignore lexical scopes (cref) referring to singleton classes

diff --git a/truffle/src/main/java/org/jruby/truffle/language/objects/ReadClassVariableNode.java b/truffle/src/main/java/org/jruby/truffle/language/objects/ReadClassVariableNode.java
@@ -37,7 +37,7 @@ public ReadClassVariableNode(RubyContext context, SourceSection sourceSection,
     public Object execute(VirtualFrame frame) {
         // TODO CS 21-Feb-16 these two operations are uncached and use loops - same for isDefined below
 
-        final DynamicObject module = lexicalScope.resolveTargetModule();
+        final DynamicObject module = lexicalScope.resolveTargetModuleForClassVariables();
 
         final Object value = ModuleOperations.lookupClassVariable(module, name);
 
@@ -51,7 +51,7 @@ public Object execute(VirtualFrame frame) {
 
     @Override
     public Object isDefined(VirtualFrame frame) {
-        final DynamicObject module = lexicalScope.resolveTargetModule();
+        final DynamicObject module = lexicalScope.resolveTargetModuleForClassVariables();
 
         final Object value = ModuleOperations.lookupClassVariable(module, name);
 

diff --git a/truffle/src/main/java/org/jruby/truffle/language/objects/WriteClassVariableNode.java b/truffle/src/main/java/org/jruby/truffle/language/objects/WriteClassVariableNode.java
@@ -38,7 +38,7 @@ public Object execute(VirtualFrame frame) {
 
         // TODO CS 21-Feb-16 these two operations are uncached and use loops
 
-        final DynamicObject module = lexicalScope.resolveTargetModule();
+        final DynamicObject module = lexicalScope.resolveTargetModuleForClassVariables();
 
         ModuleOperations.setClassVariable(getContext(), module, name, rhsValue, this);