Updated semantics in spec and RangeNodes.java

Author: Matthew Alp

spec/ruby/language/range_spec.rb

@@ -19,21 +19,25 @@
     it "is frozen" do
       (42..).should.frozen?
     end
+  end
+
+  ruby_version_is "2.7" do
+    it "creates beginless ranges" do
+      eval("(..1)").should == Range.new(nil, 1)
+      eval("(...1)").should == Range.new(nil, 1, true)
+    end
+  end
+end
 
+describe "Object Ranges" do
+  ruby_version_is "3.0" do
     it "is not frozen if it is a subclass of Range" do
-      change_range = Class.new(Range)
+      change_range = Class.new(Range).new(1, 2)
       change_range.should_not.frozen?
     end
 
     it "is not frozen if it is created through allocate" do
       Range.allocate.should_not.frozen?
     end
   end
-
-  ruby_version_is "2.7" do
-    it "creates beginless ranges" do
-      eval("(..1)").should == Range.new(nil, 1)
-      eval("(...1)").should == Range.new(nil, 1, true)
-    end
-  end
 end

src/main/java/org/truffleruby/core/range/RangeNodes.java

@@ -548,18 +548,18 @@ protected RubyLongRange longRange(RubyClass rubyClass, long begin, long end, boo
             return range;
         }
 
-        @Specialization(guards = { "nonStandardClass || (!isImplicitLong(begin) || !isImplicitLong(end))" })
+        @Specialization(guards = { "!standardClass || (!isImplicitLong(begin) || !isImplicitLong(end))" })
         protected RubyObjectRange objectRange(RubyClass rubyClass, Object begin, Object end, boolean excludeEnd,
                 @Cached DispatchNode compare,
-                @Bind("rubyClass != getRangeClass()") boolean nonStandardClass) {
+                @Bind("rubyClass == getRangeClass()") boolean standardClass) {
 
             if (compare.call(begin, "<=>", end) == nil && end != nil && begin != nil) {
                 throw new RaiseException(getContext(), coreExceptions().argumentError("bad value for range", this));
             }
 
             final Shape shape = getLanguage().objectRangeShape;
             final RubyObjectRange range = new RubyObjectRange(rubyClass, shape, excludeEnd, begin, end,
-                    !nonStandardClass);
+                    standardClass);
             AllocationTracing.trace(range, this);
             return range;
         }