Merge pull request #8282 from headius/import_methods_warning_fix

headius · web-flow · commit ab66c9e8cddb · 2024-06-04T12:45:11.000-05:00
Properly check for Module's superclass
diff --git a/core/src/main/java/org/jruby/RubyModule.java b/core/src/main/java/org/jruby/RubyModule.java
@@ -6034,7 +6034,7 @@ public static IRubyObject import_methods(ThreadContext context, IRubyObject self
 
                 RubyModule module = (RubyModule) _module;
 
-                if (module.getSuperClass() != runtime.getObject()) {
+                if (module.getSuperClass() != null) {
                     runtime.getWarnings().warn(module.getName() + " has ancestors, but Refinement#import_methods doesn't import their methods");
                 }
             }

Original file line number	Diff line number	Diff line change
`@@ -6034,7 +6034,7 @@ public static IRubyObject import_methods(ThreadContext context, IRubyObject self`
`6034`	`6034`
`6035`	`6035`	`RubyModule module = (RubyModule) _module;`
`6036`	`6036`
`6037`		`- if (module.getSuperClass() != runtime.getObject()) {`
	`6037`	`+ if (module.getSuperClass() != null) {`
`6038`	`6038`	`runtime.getWarnings().warn(module.getName() + " has ancestors, but Refinement#import_methods doesn't import their methods");`
`6039`	`6039`	`}`
`6040`	`6040`	`}`