REGRESSION (r96595): First frame in assertion backtraces is no longer labeled "1"
authorgavinp@chromium.org <gavinp@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Oct 2011 23:02:05 +0000 (23:02 +0000)
committergavinp@chromium.org <gavinp@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Oct 2011 23:02:05 +0000 (23:02 +0000)
https://bugs.webkit.org/show_bug.cgi?id=69556

Reviewed by Adam Roben.

* wtf/Assertions.cpp:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@96869 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/wtf/Assertions.cpp

index 29d8733..c61554f 100644 (file)
@@ -1,3 +1,12 @@
+2011-10-06  Gavin Peters  <gavinp@chromium.org>
+
+        REGRESSION (r96595): First frame in assertion backtraces is no longer labeled "1"
+        https://bugs.webkit.org/show_bug.cgi?id=69556
+
+        Reviewed by Adam Roben.
+
+        * wtf/Assertions.cpp:
+
 2011-10-06  Filip Pizlo  <fpizlo@apple.com>
 
         DFG implementation of UInt32ToNumber is missing a break statement
index 04a090a..e216dce 100644 (file)
@@ -211,10 +211,11 @@ void WTFReportBacktrace()
         if (mangledName)
             cxaDemangled = abi::__cxa_demangle(mangledName, 0, 0, 0);
 #endif
+        const int frameNumber = i - framesToSkip + 1;
         if (mangledName || cxaDemangled)
-            fprintf(stderr, "%-3d %p %s\n", i, samples[i], cxaDemangled ? cxaDemangled : mangledName);
+            fprintf(stderr, "%-3d %p %s\n", frameNumber, samples[i], cxaDemangled ? cxaDemangled : mangledName);
         else
-            fprintf(stderr, "%-3d %p\n", i, samples[i]);
+            fprintf(stderr, "%-3d %p\n", frameNumber, samples[i]);
         free(cxaDemangled);
     }
 }