Skip to content

Commit

Permalink
This condition does not seem necessary anymore. Fixes #3008.
Browse files Browse the repository at this point in the history
I believe this condition was added by @tdeuhr before he adjusted
the lookup logic to work like MRI. Without it, all tests and specs
appear to pass as before, and the error from #3008 goes away.
  • Loading branch information
headius committed Jun 2, 2015
1 parent 62f4721 commit 3579f2e
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -922,7 +922,7 @@ public static IRubyObject unresolvedSuper(ThreadContext context, IRubyObject sel
DynamicMethod method = superClass != null ? superClass.searchMethod(methodName) : UndefinedMethod.INSTANCE;

IRubyObject rVal = null;
if (method.isUndefined()|| (superClass.isPrepended() && (method.isImplementedBy(self.getType())))) {
if (method.isUndefined()) {
rVal = Helpers.callMethodMissing(context, self, method.getVisibility(), methodName, CallType.SUPER, args, block);
} else {
rVal = method.call(context, self, superClass, methodName, args, block);
Expand Down

0 comments on commit 3579f2e

Please sign in to comment.