Rename backtraceName to originalName to better reflect its meaning.

Co-authored-by: Ufuk Kayserilioglu <[email protected]>

Author: 2 people

src/main/java/org/truffleruby/core/MainNodes.java

@@ -73,7 +73,7 @@ protected Object mainUsing(Frame callerFrame, Object self, Object[] rubyArgs, Ro
 
         @TruffleBoundary
         private boolean isCalledFromTopLevel(InternalMethod callerMethod) {
-            final String name = callerMethod.getSharedMethodInfo().getBacktraceName();
+            final String name = callerMethod.getSharedMethodInfo().getOriginalName();
             return name.equals("<main>") || name.startsWith("<top ");
         }
     }

src/main/java/org/truffleruby/core/method/UnboundMethodNodes.java

@@ -153,7 +153,7 @@ public abstract static class OriginalNameNode extends CoreMethodArrayArgumentsNo
         @Specialization
         protected RubySymbol originalName(RubyUnboundMethod unboundMethod,
                 @Cached ToSymbolNode toSymbolNode) {
-            String originalName = unboundMethod.method.getSharedMethodInfo().getBacktraceName();
+            String originalName = unboundMethod.method.getSharedMethodInfo().getOriginalName();
 
             return toSymbolNode.execute(originalName);
         }

src/main/java/org/truffleruby/core/proc/ProcNodes.java

@@ -286,7 +286,7 @@ public abstract static class ProcSymbolToProcSymbolNode extends PrimitiveArrayAr
         @Specialization
         protected Object symbolToProcSymbol(RubyProc proc) {
             if (proc.arity == SymbolNodes.ToProcNode.ARITY) {
-                return getSymbol(proc.getSharedMethodInfo().getBacktraceName());
+                return getSymbol(proc.getSharedMethodInfo().getOriginalName());
             } else {
                 return nil;
             }

src/main/java/org/truffleruby/language/backtrace/Backtrace.java

@@ -151,7 +151,7 @@ public static String labelFor(TruffleStackTraceElement e) {
         RootNode root = e.getTarget().getRootNode();
         String label = root instanceof RubyRootNode
                 // Ruby backtraces do not include the class name for MRI compatibility.
-                ? ((RubyRootNode) root).getSharedMethodInfo().getBacktraceName()
+                ? ((RubyRootNode) root).getSharedMethodInfo().getOriginalName()
                 : root.getName();
         return label == null ? "<unknown>" : label;
     }

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

@@ -34,7 +34,7 @@ public final class SharedMethodInfo {
     private final Arity arity;
     /** The original name of the method. Does not change when aliased. Looks like "block in foo" or "block (2 levels) in
      * foo" for blocks. This is the name shown in backtraces: "from FILE:LINE:in `NAME'". */
-    private final String backtraceName;
+    private final String originalName;
     /** The "static" name of this method at parse time, such as "M::C#foo", "M::C.foo", "<module:Inner>", "block (2
      * levels) in M::C.foo" or "block (2 levels) in <module:Inner>". This name is used for tools. */
     private final String parseName;
@@ -48,16 +48,16 @@ public SharedMethodInfo(
             SourceSection sourceSection,
             LexicalScope staticLexicalScope,
             Arity arity,
-            String backtraceName,
+            String originalName,
             int blockDepth,
             String parseName,
             String notes,
             ArgumentDescriptor[] argumentDescriptors) {
-        assert blockDepth == 0 || backtraceName.startsWith("block ") : backtraceName;
+        assert blockDepth == 0 || originalName.startsWith("block ") : originalName;
         this.sourceSection = sourceSection;
         this.staticLexicalScope = staticLexicalScope;
         this.arity = arity;
-        this.backtraceName = backtraceName;
+        this.originalName = originalName;
         this.blockDepth = blockDepth;
         this.parseName = parseName;
         this.notes = notes;
@@ -93,7 +93,7 @@ public SharedMethodInfo withArity(Arity newArity) {
                 sourceSection,
                 staticLexicalScope,
                 newArity,
-                backtraceName,
+                originalName,
                 blockDepth,
                 parseName,
                 notes,
@@ -131,7 +131,7 @@ public boolean isBlock() {
 
     @TruffleBoundary
     public boolean isModuleBody() {
-        boolean isModuleBody = isModuleBody(getBacktraceName());
+        boolean isModuleBody = isModuleBody(getOriginalName());
         assert !(isModuleBody && isBlock()) : this;
         return isModuleBody;
     }
@@ -150,19 +150,19 @@ public static boolean isModuleBody(String name) {
         }
     }
 
-    public String getBacktraceName() {
-        return backtraceName;
+    public String getOriginalName() {
+        return originalName;
     }
 
     /** Returns the method name on its own. Can start with "<" like "<module:Inner>" for module bodies. */
     public String getMethodName() {
-        return blockDepth == 0 ? backtraceName : notes;
+        return blockDepth == 0 ? originalName : notes;
     }
 
     /** More efficient than {@link #getMethodName()} when we know blockDepth == 0 */
     public String getMethodNameForNotBlock() {
         assert blockDepth == 0;
-        return backtraceName;
+        return originalName;
     }
 
     public String getParseName() {

src/main/java/org/truffleruby/parser/BodyTranslator.java

@@ -1079,7 +1079,7 @@ private ModuleBodyDefinition compileClassNode(SourceIndexLength sourceSection, P
                 environment.getReturnID());
 
         return new ModuleBodyDefinition(
-                environment.getSharedMethodInfo().getBacktraceName(),
+                environment.getSharedMethodInfo().getOriginalName(),
                 environment.getSharedMethodInfo(),
                 rootNode.getCallTarget(),
                 environment.getStaticLexicalScopeOrNull());
@@ -2006,14 +2006,14 @@ private RubyNode translateBlockLikeNode(IterParseNode node, boolean isStabbyLamb
         final int blockDepth = environment.getBlockDepth() + 1;
 
         // "block in foo"
-        String backtraceName = SharedMethodInfo.getBlockName(blockDepth, methodName);
+        String originalName = SharedMethodInfo.getBlockName(blockDepth, methodName);
         // "block (2 levels) in M::C.foo"
         String parseName = SharedMethodInfo.getBlockName(blockDepth, methodParent.getSharedMethodInfo().getParseName());
         final SharedMethodInfo sharedMethodInfo = new SharedMethodInfo(
                 sourceSection.toSourceSection(source),
                 environment.getStaticLexicalScopeOrNull(),
                 argsNode.getArity(),
-                backtraceName,
+                originalName,
                 blockDepth,
                 parseName,
                 methodName,