RESOLVED FIXED Bug 58405
BACKTRACE() macro should check for Debug configuration in macro, not WTFReportBacktrace definition 
https://bugs.webkit.org/show_bug.cgi?id=58405
Summary BACKTRACE() macro should check for Debug configuration in macro, not WTFRepor...
Paul Knight
Reported 2011-04-12 17:53:31 PDT
The BACKTRACE() macro currently checks whether NDEBUG is defined in the implementation of WTFReportBacktrace, and not in the BACKTRACE() macro. This means you need a Debug JavaScriptCore in order for it to do anything. It seems the check should be moved to the header file, so you can use BACKTRACE() in a WebCore debug build and not need a Debug JavaScriptCore, similar to the way the other macros like ASSERT() work.
Attachments
Proposed patch (2.17 KB, patch)
2011-04-12 21:53 PDT, Paul Knight 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Paul Knight
Comment 1 2011-04-12 21:53:10 PDT
Created attachment 89337 [details] Proposed patch
Gavin Barraclough
Comment 2 2011-04-12 22:10:24 PDT
Comment on attachment 89337 [details] Proposed patch Looks great.
WebKit Commit Bot
Comment 3 2011-04-13 01:14:56 PDT
Comment on attachment 89337 [details] Proposed patch Clearing flags on attachment: 89337 Committed r83709: <http://trac.webkit.org/changeset/83709>
WebKit Commit Bot
Comment 4 2011-04-13 01:15:01 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.