Changeset 39600 in webkit for trunk/JavaScriptCore/jit

Timestamp:

Jan 5, 2009, 9:21:13 AM (16 years ago)

Author:

[email protected]

Message:

Reviewed by Darin Adler.

​https://bugs.webkit.org/show_bug.cgi?id=23115
Create a version of ASSERT for use with otherwise unused variables

JavaScriptCore:

• wtf/Assertions.h: Added ASSERT_UNUSED.

• jit/ExecutableAllocatorPosix.cpp: (JSC::ExecutablePool::systemRelease):
• runtime/Collector.cpp: (JSC::Heap::destroy): (JSC::Heap::heapAllocate):
• runtime/JSNotAnObject.cpp: (JSC::JSNotAnObject::toPrimitive): (JSC::JSNotAnObject::getPrimitiveNumber): (JSC::JSNotAnObject::toBoolean): (JSC::JSNotAnObject::toNumber): (JSC::JSNotAnObject::toString): (JSC::JSNotAnObject::getOwnPropertySlot): (JSC::JSNotAnObject::put): (JSC::JSNotAnObject::deleteProperty): (JSC::JSNotAnObject::getPropertyNames):
• wtf/TCSystemAlloc.cpp: (TCMalloc_SystemRelease): Use it in some places that used other idioms for this purpose.

WebCore:

• loader/appcache/ApplicationCacheStorage.cpp: (WebCore::ApplicationCacheStorage::verifySchemaVersion): Use the new ASSERT_UNUSED macro.

File:

• trunk/JavaScriptCore/jit/ExecutableAllocatorPosix.cpp (modified) (1 diff)

trunk/JavaScriptCore/jit/ExecutableAllocatorPosix.cpp

@@ -48 +48 @@
 void ExecutablePool::systemRelease(const ExecutablePool::Allocation& alloc) 
 { 
-    if (munmap(alloc.pages, alloc.size))
-        ASSERT_NOT_REACHED();
+    int result = munmap(alloc.pages, alloc.size);
+    ASSERT_UNUSED(result, !result);
 }