[Qt] Fix crashes with package builds in release
authorhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 9 Apr 2010 09:03:15 +0000 (09:03 +0000)
committerhausmann@webkit.org <hausmann@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 9 Apr 2010 09:03:15 +0000 (09:03 +0000)
Patch by Jocelyn Turcotte <jocelyn.turcotte@nokia.com> on 2010-04-09
Reviewed by Simon Hausmann.

.:

* WebKit.pri: Don't randomly add NDEBUG to the defines.

JavaScriptCore:

Add NDEBUG to the defines for package build in JavaScriptCore.pri,
so that it's consistently used for JavaScriptCore, WebCore, jsc and all
other tools using wtf, etc. data structures directly. Mixing NDEBUG with
non-NDEBUG builds causes crashes due to differences in data structures when
assertions/checks are enabled.

* JavaScriptCore.pri:

WebCore:

* WebCore.pro: Don't add NDEBUG to the defines here, add it from JavaScriptCore.pri

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

ChangeLog
JavaScriptCore/ChangeLog
JavaScriptCore/JavaScriptCore.pri
WebCore/ChangeLog
WebCore/WebCore.pro
WebKit.pri

index 9b6cbac..0eb75fa 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2010-04-09  Jocelyn Turcotte  <jocelyn.turcotte@nokia.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] Fix crashes with package builds in release
+
+        * WebKit.pri: Don't randomly add NDEBUG to the defines.
+
 2010-04-07  Laszlo Gombos  <laszlo.1.gombos@nokia.com>
 
         Reviewed by Kenneth Rohde Christiansen.
index e781a8f..0b2b8e7 100644 (file)
@@ -1,3 +1,17 @@
+2010-04-09  Jocelyn Turcotte  <jocelyn.turcotte@nokia.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] Fix crashes with package builds in release
+
+        Add NDEBUG to the defines for package build in JavaScriptCore.pri,
+        so that it's consistently used for JavaScriptCore, WebCore, jsc and all
+        other tools using wtf, etc. data structures directly. Mixing NDEBUG with
+        non-NDEBUG builds causes crashes due to differences in data structures when
+        assertions/checks are enabled.
+
+        * JavaScriptCore.pri:
+
 2010-04-09  Patrick Gansterer  <paroga@paroga.com>
 
         Reviewed by Darin Adler.
index 1ad0251..c55c48b 100644 (file)
@@ -8,6 +8,8 @@ CONFIG(standalone_package) {
     isEmpty(JSC_GENERATED_SOURCES_DIR):JSC_GENERATED_SOURCES_DIR = generated
 }
 
+CONFIG(standalone_package): DEFINES *= NDEBUG
+
 symbian: {
     # Need to guarantee this comes before system includes of /epoc32/include
     MMP_RULES += "USERINCLUDE ../JavaScriptCore/profiler"
index 83583bf..027d2c6 100644 (file)
@@ -1,3 +1,11 @@
+2010-04-09  Jocelyn Turcotte  <jocelyn.turcotte@nokia.com>
+
+        Reviewed by Simon Hausmann.
+
+        [Qt] Fix crashes with package builds in release
+
+        * WebCore.pro: Don't add NDEBUG to the defines here, add it from JavaScriptCore.pri
+
 2010-04-09  Kenneth Rohde Christiansen  <kenneth@webkit.org>
 
         Reviewed by Simon Fraser.
index e7bb983..14073fd 100644 (file)
@@ -52,7 +52,6 @@ CONFIG(standalone_package) {
     isEmpty(JSC_GENERATED_SOURCES_DIR):JSC_GENERATED_SOURCES_DIR = $$PWD/../JavaScriptCore/generated
 
     PRECOMPILED_HEADER = $$PWD/../WebKit/qt/WebKit_pch.h
-    DEFINES *= NDEBUG
 
     symbian: TARGET += $${QT_LIBINFIX}
 } else {
index 4f425bb..8877437 100644 (file)
@@ -43,10 +43,6 @@ greaterThan(QT_MINOR_VERSION, 5):DEFINES += WTF_USE_ACCELERATED_COMPOSITING
     DEFINES += USE_SYSTEM_MALLOC
 }
 
-CONFIG(release, debug|release) {
-    DEFINES += NDEBUG
-}
-
 BASE_DIR = $$PWD
 INCLUDEPATH += $$OUTPUT_DIR/include/QtWebKit