Skip to content

Commit 910ef3e

Browse files
committed
MethodBuiltins refactoring to reuse the special name constants
1 parent f235752 commit 910ef3e

File tree

1 file changed

+12
-9
lines changed
  • graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/method

1 file changed

+12
-9
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/method/MethodBuiltins.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -26,16 +26,19 @@
2626

2727
package com.oracle.graal.python.builtins.objects.method;
2828

29+
import static com.oracle.graal.python.nodes.SpecialAttributeNames.__CODE__;
30+
import static com.oracle.graal.python.nodes.SpecialAttributeNames.__FUNC__;
31+
import static com.oracle.graal.python.nodes.SpecialAttributeNames.__NAME__;
32+
import static com.oracle.graal.python.nodes.SpecialAttributeNames.__SELF__;
2933
import static com.oracle.graal.python.nodes.SpecialMethodNames.__CALL__;
34+
import static com.oracle.graal.python.nodes.SpecialMethodNames.__EQ__;
3035

3136
import java.util.List;
3237

3338
import com.oracle.graal.python.builtins.Builtin;
3439
import com.oracle.graal.python.builtins.CoreFunctions;
3540
import com.oracle.graal.python.builtins.PythonBuiltins;
3641
import com.oracle.graal.python.builtins.objects.function.PKeyword;
37-
import com.oracle.graal.python.nodes.SpecialAttributeNames;
38-
import com.oracle.graal.python.nodes.SpecialMethodNames;
3942
import com.oracle.graal.python.nodes.argument.CreateArgumentsNode;
4043
import com.oracle.graal.python.nodes.call.CallDispatchNode;
4144
import com.oracle.graal.python.nodes.call.special.LookupAndCallBinaryNode;
@@ -73,7 +76,7 @@ protected Object doIt(PBuiltinMethod self, Object[] arguments, PKeyword[] keywor
7376
}
7477
}
7578

76-
@Builtin(name = "__self__", fixedNumOfArguments = 1, isGetter = true)
79+
@Builtin(name = __SELF__, fixedNumOfArguments = 1, isGetter = true)
7780
@GenerateNodeFactory
7881
public abstract static class SelfNode extends PythonBuiltinNode {
7982
@Specialization
@@ -87,7 +90,7 @@ protected Object doIt(PBuiltinMethod self) {
8790
}
8891
}
8992

90-
@Builtin(name = "__func__", fixedNumOfArguments = 1, isGetter = true)
93+
@Builtin(name = __FUNC__, fixedNumOfArguments = 1, isGetter = true)
9194
@GenerateNodeFactory
9295
public abstract static class FuncNode extends PythonBuiltinNode {
9396
@Specialization
@@ -101,13 +104,13 @@ protected Object doIt(PBuiltinMethod self) {
101104
}
102105
}
103106

104-
@Builtin(name = SpecialAttributeNames.__NAME__, fixedNumOfArguments = 1, isGetter = true)
107+
@Builtin(name = __NAME__, fixedNumOfArguments = 1, isGetter = true)
105108
@GenerateNodeFactory
106109
public abstract static class NameNode extends PythonBuiltinNode {
107110
@Specialization
108111
protected Object doIt(PMethod self,
109112
@Cached("create(__GETATTRIBUTE__)") LookupAndCallBinaryNode getCode) {
110-
return getCode.executeObject(self.getFunction(), SpecialAttributeNames.__NAME__);
113+
return getCode.executeObject(self.getFunction(), __NAME__);
111114
}
112115

113116
@Specialization
@@ -116,17 +119,17 @@ protected Object doIt(PBuiltinMethod self) {
116119
}
117120
}
118121

119-
@Builtin(name = SpecialAttributeNames.__CODE__, fixedNumOfArguments = 1, isGetter = true)
122+
@Builtin(name = __CODE__, fixedNumOfArguments = 1, isGetter = true)
120123
@GenerateNodeFactory
121124
public abstract static class CodeNode extends PythonBuiltinNode {
122125
@Specialization
123126
protected Object doIt(PMethod self,
124127
@Cached("create(__GETATTRIBUTE__)") LookupAndCallBinaryNode getCode) {
125-
return getCode.executeObject(self.getFunction(), SpecialAttributeNames.__CODE__);
128+
return getCode.executeObject(self.getFunction(), __CODE__);
126129
}
127130
}
128131

129-
@Builtin(name = SpecialMethodNames.__EQ__, fixedNumOfArguments = 2)
132+
@Builtin(name = __EQ__, fixedNumOfArguments = 2)
130133
@GenerateNodeFactory
131134
abstract static class EqNode extends PythonBinaryBuiltinNode {
132135
@Specialization

0 commit comments

Comments
 (0)