From d2458e66c26bac5e9db98c22dc6910cb45971b85 Mon Sep 17 00:00:00 2001
From: "Yukihiro \"Matz\" Matsumoto" <matz@ruby-lang.org>
Date: Tue, 23 May 2017 17:57:28 +0900
Subject: Remove class info from backtrace lines.

---
 src/backtrace.c | 45 ++-------------------------------------------
 src/gc.c        |  4 ----
 2 files changed, 2 insertions(+), 47 deletions(-)

(limited to 'src')

diff --git a/src/backtrace.c b/src/backtrace.c
index b5f8e77c9..75f926b2e 100644
--- a/src/backtrace.c
+++ b/src/backtrace.c
@@ -19,8 +19,6 @@ struct backtrace_location_raw {
   int lineno;
   const char *filename;
   mrb_sym method_id;
-  char sep;
-  struct RClass *klass;
 };
 
 struct backtrace_location {
@@ -28,8 +26,6 @@ struct backtrace_location {
   int lineno;
   const char *filename;
   const char *method;
-  char sep;
-  const char *class_name;
 };
 
 typedef void (*each_backtrace_func)(mrb_state*, struct backtrace_location_raw*, void*);
@@ -57,12 +53,7 @@ print_backtrace_i(mrb_state *mrb, struct backtrace_location *loc, void *data)
   fprintf(args->stream, "\t[%d] %s:%d", loc->i, loc->filename, loc->lineno);
 
   if (loc->method) {
-    if (loc->class_name) {
-      fprintf(args->stream, ":in %s%c%s", loc->class_name, loc->sep, loc->method);
-    }
-    else {
-      fprintf(args->stream, ":in %s", loc->method);
-    }
+    fprintf(args->stream, ":in %s", loc->method);
   }
 
   fprintf(args->stream, "\n");
@@ -83,12 +74,6 @@ get_backtrace_i(mrb_state *mrb, struct backtrace_location *loc, void *data)
 
   if (loc->method) {
     mrb_str_cat_lit(mrb, str, ":in ");
-
-    if (loc->class_name) {
-      mrb_str_cat_cstr(mrb, str, loc->class_name);
-      mrb_str_cat(mrb, str, &loc->sep, 1);
-    }
-
     mrb_str_cat_cstr(mrb, str, loc->method);
   }
 
@@ -132,19 +117,11 @@ each_backtrace(mrb_state *mrb, mrb_int ciidx, mrb_code *pc0, each_backtrace_func
 
     if (loc.lineno == -1) continue;
 
-    if (ci->target_class == ci->proc->target_class) {
-      loc.sep = '.';
-    }
-    else {
-      loc.sep = '#';
-    }
-
     if (!loc.filename) {
       loc.filename = "(unknown)";
     }
 
     loc.method_id = ci->mid;
-    loc.klass = ci->proc->target_class;
     loc.i = i;
     func(mrb, &loc, data);
   }
@@ -165,8 +142,6 @@ output_backtrace_i(mrb_state *mrb, struct backtrace_location_raw *loc_raw, void
   loc.lineno     = loc_raw->lineno;
   loc.filename   = loc_raw->filename;
   loc.method     = mrb_sym2name(mrb, loc_raw->method_id);
-  loc.sep        = loc_raw->sep;
-  loc.class_name = mrb_class_name(mrb, loc_raw->klass);
 
   args->func(mrb, &loc, args->data);
 }
@@ -244,15 +219,7 @@ print_backtrace_saved(mrb_state *mrb)
       const char *method_name;
 
       method_name = mrb_sym2name(mrb, entry->method_id);
-      if (entry->klass) {
-        fprintf(stream, ":in %s%c%s",
-                mrb_class_name(mrb, entry->klass),
-                entry->sep,
-                method_name);
-      }
-      else {
-        fprintf(stream, ":in %s", method_name);
-      }
+      fprintf(stream, ":in %s", method_name);
       mrb_gc_arena_restore(mrb, ai);
     }
 
@@ -353,8 +320,6 @@ save_backtrace_i(mrb_state *mrb,
   entry = &mrb->backtrace.entries[mrb->backtrace.n];
   entry->filename  = loc_raw->filename;
   entry->lineno    = loc_raw->lineno;
-  entry->klass     = loc_raw->klass;
-  entry->sep       = loc_raw->sep;
   entry->method_id = loc_raw->method_id;
 
   mrb->backtrace.n++;
@@ -408,12 +373,6 @@ mrb_restore_backtrace(mrb_state *mrb)
                            mrb_fixnum_value(entry->lineno));
     if (entry->method_id != 0) {
       mrb_str_cat_lit(mrb, mrb_entry, ":in ");
-
-      if (entry->klass) {
-        mrb_str_cat_cstr(mrb, mrb_entry, mrb_class_name(mrb, entry->klass));
-        mrb_str_cat(mrb, mrb_entry, &entry->sep, 1);
-      }
-
       mrb_str_cat_cstr(mrb, mrb_entry, mrb_sym2name(mrb, entry->method_id));
     }
 
diff --git a/src/gc.c b/src/gc.c
index 0fff408f3..bd633a9fe 100644
--- a/src/gc.c
+++ b/src/gc.c
@@ -872,10 +872,6 @@ root_scan_phase(mrb_state *mrb, mrb_gc *gc)
   mrb_gc_mark(mrb, (struct RBasic*)mrb->exc);
   /* mark backtrace */
   mrb_gc_mark(mrb, (struct RBasic*)mrb->backtrace.exc);
-  e = (size_t)mrb->backtrace.n;
-  for (i=0; i<e; i++) {
-    mrb_gc_mark(mrb, (struct RBasic*)mrb->backtrace.entries[i].klass);
-  }
   /* mark pre-allocated exception */
   mrb_gc_mark(mrb, (struct RBasic*)mrb->nomem_err);
   mrb_gc_mark(mrb, (struct RBasic*)mrb->stack_err);
-- 
cgit v1.2.3