Skip to content

Commit bc20ac6

Browse files
committed
Make GetMethodObjectNode#execute public
* Now that we found out that having a public execute(Frame, ...) is fine.
1 parent 2d95cfe commit bc20ac6

File tree

3 files changed

+4
-12
lines changed

3 files changed

+4
-12
lines changed

src/main/java/org/truffleruby/core/kernel/KernelNodes.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1310,8 +1310,7 @@ protected RubyNode coerceToString(RubyNode name) {
13101310

13111311
@Specialization
13121312
protected RubyMethod method(VirtualFrame frame, Object self, Object name) {
1313-
return getMethodObjectNode
1314-
.get(frame, self, name, PRIVATE, readCallerFrame.execute(frame));
1313+
return getMethodObjectNode.execute(frame, self, name, PRIVATE, readCallerFrame.execute(frame));
13151314
}
13161315

13171316
}
@@ -1453,8 +1452,7 @@ protected RubyNode coerceToString(RubyNode name) {
14531452

14541453
@Specialization
14551454
protected RubyMethod publicMethod(VirtualFrame frame, Object self, Object name) {
1456-
return getMethodObjectNode
1457-
.get(frame, self, name, PUBLIC, readCallerFrame.execute(frame));
1455+
return getMethodObjectNode.execute(frame, self, name, PUBLIC, readCallerFrame.execute(frame));
14581456
}
14591457

14601458
}

src/main/java/org/truffleruby/language/RubyDynamicObject.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -375,8 +375,7 @@ public Object readMember(String name,
375375
if (ivarFoundProfile.profile(iVar != null)) {
376376
return iVar;
377377
} else if (definedNode.execute(null, this, name)) {
378-
return getMethodObjectNode
379-
.get(null, this, rubyName, DispatchConfiguration.PRIVATE, null);
378+
return getMethodObjectNode.execute(null, this, rubyName, DispatchConfiguration.PRIVATE, null);
380379
} else {
381380
errorProfile.enter();
382381
throw UnknownIdentifierException.create(name);

src/main/java/org/truffleruby/language/methods/GetMethodObjectNode.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,12 +47,7 @@ public static GetMethodObjectNode create() {
4747
return GetMethodObjectNodeGen.create();
4848
}
4949

50-
public RubyMethod get(VirtualFrame frame, Object self, Object name,
51-
DispatchConfiguration config, MaterializedFrame callerFrame) {
52-
return execute(frame, self, name, config, callerFrame);
53-
}
54-
55-
protected abstract RubyMethod execute(Frame frame, Object self, Object name,
50+
public abstract RubyMethod execute(Frame frame, Object self, Object name,
5651
DispatchConfiguration dispatchConfig, MaterializedFrame callerFrame);
5752

5853
@Specialization

0 commit comments

Comments
 (0)