Minor cleanup

Author: andrykonchin

spec/ruby/core/module/const_added_spec.rb

@@ -122,7 +122,9 @@ class SubClass
       ScratchPad.recorded.should == [line + 2, line + 4, line + 7, line + 11]
     end
 
-    it "is called when the constant is ready to be used" do
+    it "is called when the constant is already assigned a value" do
+      ScratchPad.record []
+
       mod = Module.new do
         def self.const_added(name)
           ScratchPad.record const_get(name)

spec/truffleruby.next-specs

@@ -30,8 +30,8 @@ spec/ruby/core/sizedqueue/deq_spec.rb
 spec/ruby/core/sizedqueue/pop_spec.rb
 spec/ruby/core/sizedqueue/shift_spec.rb
 
+spec/ruby/core/module/const_added_spec.rb
 spec/ruby/core/module/refinements_spec.rb
 spec/ruby/core/refinement/refined_class_spec.rb
 spec/ruby/core/module/used_refinements_spec.rb
 spec/ruby/optional/capi/hash_spec.rb
-spec/ruby/core/module/const_added_spec.rb

src/main/java/org/truffleruby/core/module/ModuleFields.java

@@ -458,6 +458,7 @@ public void setAutoloadConstant(RubyContext context, Node currentNode, String na
         context.getFeatureLoader().addAutoload(autoloadConstant);
     }
 
+    @TruffleBoundary
     private RubyConstant setConstantInternal(RubyContext context, Node currentNode, String name, Object value,
             boolean autoload) {
         checkFrozen(context, currentNode);
@@ -497,8 +498,8 @@ private RubyConstant setConstantInternal(RubyContext context, Node currentNode,
 
         final CoreLibrary coreLibrary = context.getCoreLibrary();
         if (currentNode != null && coreLibrary != null && coreLibrary.isLoaded()) {
-            final RubySymbol constSymbol = context.getLanguageSlow().getSymbol(name);
-            RubyContext.send(currentNode, rubyModule, "const_added", constSymbol);
+            final RubySymbol nameSymbol = context.getLanguageSlow().getSymbol(name);
+            RubyContext.send(currentNode, rubyModule, "const_added", nameSymbol);
         }
 
         return autoload ? newConstant : previous;

src/main/ruby/truffleruby/core/module.rb

@@ -61,6 +61,9 @@ def include?(mod)
     ancestors.any? { |m| Primitive.equal?(mod, m) }
   end
 
+  private def const_added(name)
+  end
+
   private def method_added(name)
   end
 
@@ -105,9 +108,6 @@ def prepend(*modules)
     self
   end
 
-  private def const_added(name)
-  end
-
   def const_defined?(name, inherit = true)
     Primitive.module_const_defined?(self, name, inherit, true)
   end