Reviewed by Darin Adler.
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jan 2010 19:42:07 +0000 (19:42 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Jan 2010 19:42:07 +0000 (19:42 +0000)
        https://bugs.webkit.org/show_bug.cgi?id=33540
        Make it possible to build in debug mode with assertions disabled

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

14 files changed:
JavaScriptCore/ChangeLog
JavaScriptCore/jit/JITStubs.cpp
JavaScriptCore/runtime/Identifier.cpp
JavaScriptCore/wtf/FastMalloc.cpp
JavaScriptCore/wtf/HashTable.h
JavaScriptCore/yarr/RegexCompiler.cpp
WebCore/ChangeLog
WebCore/dom/Element.cpp
WebCore/dom/SelectElement.cpp
WebCore/editing/TextIterator.cpp
WebCore/loader/Cache.cpp
WebCore/rendering/RenderArena.cpp
WebKitTools/ChangeLog
WebKitTools/DumpRenderTree/mac/ObjCController.m

index e9009d0..53ef271 100644 (file)
@@ -1,3 +1,16 @@
+2010-01-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=33540
+        Make it possible to build in debug mode with assertions disabled
+
+        * jit/JITStubs.cpp: (JSC::DEFINE_STUB_FUNCTION):
+        * runtime/Identifier.cpp: (JSC::Identifier::checkSameIdentifierTable):
+        * wtf/FastMalloc.cpp:
+        * wtf/HashTable.h: (WTF::HashTableConstIterator::checkValidity):
+        * yarr/RegexCompiler.cpp: (JSC::Yarr::compileRegex):
+
 2009-11-23  Yong Li  <yoli@rim.com>
 
         Reviewed by Adam Treat.
index 351c70c..b79dc84 100644 (file)
@@ -1622,7 +1622,7 @@ DEFINE_STUB_FUNCTION(void*, op_call_JSFunction)
 {
     STUB_INIT_STACK_FRAME(stackFrame);
 
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     CallData callData;
     ASSERT(stackFrame.args[0].jsValue().getCallData(callData) == CallTypeJS);
 #endif
@@ -1871,7 +1871,7 @@ DEFINE_STUB_FUNCTION(JSObject*, op_construct_JSConstruct)
         VM_THROW_EXCEPTION();
     }
 
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     ConstructData constructData;
     ASSERT(constructor->getConstructData(constructData) == ConstructTypeJS);
 #endif
index 1cfe7a3..a25c0cc 100644 (file)
@@ -247,12 +247,12 @@ void Identifier::remove(UString::Rep* r)
 
 void Identifier::checkSameIdentifierTable(ExecState* exec, UString::Rep*)
 {
-    ASSERT(exec->globalData().identifierTable == currentIdentifierTable());
+    ASSERT_UNUSED(exec, exec->globalData().identifierTable == currentIdentifierTable());
 }
 
 void Identifier::checkSameIdentifierTable(JSGlobalData* globalData, UString::Rep*)
 {
-    ASSERT(globalData->identifierTable == currentIdentifierTable());
+    ASSERT_UNUSED(globalData, globalData->identifierTable == currentIdentifierTable());
 }
 
 #else
index f943b7e..c834212 100644 (file)
@@ -114,11 +114,13 @@ static void initializeIsForbiddenKey()
   pthread_key_create(&isForbiddenKey, 0);
 }
 
+#if !ASSERT_DISABLED
 static bool isForbidden()
 {
     pthread_once(&isForbiddenKeyOnce, initializeIsForbiddenKey);
     return !!pthread_getspecific(isForbiddenKey);
 }
+#endif
 
 void fastMallocForbid()
 {
index 3b283f8..92533fa 100644 (file)
@@ -197,7 +197,7 @@ namespace WTF {
         void checkValidity(const const_iterator& other) const
         {
             ASSERT(m_table);
-            ASSERT(other.m_table);
+            ASSERT_UNUSED(other, other.m_table);
             ASSERT(m_table == other.m_table);
         }
 #else
index c7b3c81..9cd3d12 100644 (file)
@@ -708,7 +708,7 @@ const char* compileRegex(const UString& patternString, RegexPattern& pattern)
         unsigned numSubpatterns = pattern.m_numSubpatterns;
 
         constructor.reset();
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
         const char* error =
 #endif
             parse(constructor, patternString, numSubpatterns);
index e51dc0f..ed40e3f 100644 (file)
@@ -1,3 +1,16 @@
+2010-01-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=33540
+        Make it possible to build in debug mode with assertions disabled
+
+        * dom/Element.cpp: (WebCore::Element::getURLAttribute):
+        * dom/SelectElement.cpp: (WebCore::SelectElementData::checkListItems):
+        * editing/TextIterator.cpp:
+        * loader/Cache.cpp: (WebCore::Cache::removeFromLRUList):
+        * rendering/RenderArena.cpp: (WebCore::RenderArena::free):
+
 2010-01-12  Eric Carlson  <eric.carlson@apple.com>
 
         Reviewed by Darin Adler and Simon Fraser.
index 48ebf6c..c3b15f5 100644 (file)
@@ -1444,7 +1444,7 @@ bool Element::webkitMatchesSelector(const String& selector, ExceptionCode& ec)
 
 KURL Element::getURLAttribute(const QualifiedName& name) const
 {
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     if (namedAttrMap) {
         if (Attribute* attribute = namedAttrMap->getAttributeItem(name))
             ASSERT(isURLAttribute(attribute));
index 0e84a09..4f7712d 100644 (file)
@@ -957,7 +957,7 @@ SelectElementData::SelectElementData()
 
 void SelectElementData::checkListItems(const Element* element) const
 {
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     const Vector<Element*>& items = m_listItems;
     SelectElement::recalcListItems(*const_cast<SelectElementData*>(this), element, false);
     ASSERT(items == m_listItems);
index 44a203f..923f537 100644 (file)
@@ -156,7 +156,7 @@ static inline Node* parentCrossingShadowBoundaries(Node* node)
     return node->shadowParentNode();
 }
 
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
 
 static unsigned depthCrossingShadowBoundaries(Node* node)
 {
index a19fb98..0b976b8 100644 (file)
@@ -466,13 +466,13 @@ void Cache::removeFromLRUList(CachedResource* resource)
     if (resource->accessCount() == 0)
         return;
 
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     unsigned oldListIndex = resource->m_lruIndex;
 #endif
 
     LRUList* list = lruListFor(resource);
 
-#ifndef NDEBUG
+#if !ASSERT_DISABLED
     // Verify that the list we got is the list we want.
     ASSERT(resource->m_lruIndex == oldListIndex);
 
index 34325d4..1dafb2f 100644 (file)
@@ -114,7 +114,7 @@ void RenderArena::free(size_t size, void* ptr)
     // Use standard free so that memory debugging tools work.
     RenderArenaDebugHeader* header = static_cast<RenderArenaDebugHeader*>(ptr) - 1;
     ASSERT(header->signature == signature);
-    ASSERT(header->size == size);
+    ASSERT_UNUSED(size, header->size == size);
     ASSERT(header->arena == this);
     header->signature = signatureDead;
     ::free(header);
index 2f2d28c..20fb838 100644 (file)
@@ -1,3 +1,12 @@
+2010-01-12  Alexey Proskuryakov  <ap@apple.com>
+
+        Reviewed by Darin Adler.
+
+        https://bugs.webkit.org/show_bug.cgi?id=33540
+        Make it possible to build in debug mode with assertions disabled
+
+        * DumpRenderTree/mac/ObjCController.m: (-[ObjCController accessStoredWebScriptObject]):
+
 2010-01-11  Darin Fisher  <darin@chromium.org>
 
         Reviewed by Dimitri Glazkov.
index d7cc6a0..aa9ee49 100644 (file)
@@ -209,10 +209,9 @@ static void* runJavaScriptThread(void* arg)
 {
 #if !ASSERT_DISABLED
     BOOL isWindowObject = [storedWebScriptObject isKindOfClass:[DOMAbstractView class]];
-#endif
     JSObjectRef jsObject = [storedWebScriptObject JSObject];
     ASSERT((jsObject && isWindowObject) || (!jsObject && !isWindowObject));
-
+#endif
     [storedWebScriptObject callWebScriptMethod:@"" withArguments:nil];
     [storedWebScriptObject evaluateWebScript:@""];
     [storedWebScriptObject setValue:[WebUndefined undefined] forKey:@"key"];