[Webkit-unassigned] [Bug 58405] New: BACKTRACE() macro should check for Debug configuration in macro, not WTFReportBacktrace definition

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=58405

           Summary: BACKTRACE() macro should check for Debug configuration
                    in macro, not WTFReportBacktrace definition
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: PC
        OS/Version: Mac OS X 10.5
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Web Template Framework
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: pknight at apple.com


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.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.