Changeset 39809 in webkit for trunk/JavaScriptCore/wtf/TCSystemAlloc.cpp

Timestamp:

Jan 11, 2009, 1:58:23 PM (16 years ago)

Author:

Darin Adler

Message:

2009-01-11 Darin Adler <Darin Adler>

Reviewed by Dan Bernstein.

Bug 23239: improve handling of unused arguments in JavaScriptCore
​https://bugs.webkit.org/show_bug.cgi?id=23239

• runtime/DatePrototype.cpp: Moved LocaleDateTimeFormat enum outside #if so we can use this on all platforms. Changed valueOf to share the same function with getTime, since the contents of the two are identical. Removed a FIXME since the idea isn't really specific enough or helpful enough to need to sit here in the source code. (JSC::formatLocaleDate): Changed the Mac version of this function to take the same arguments as the non-Mac version so the caller doesn't have to special-case the two platforms. Also made the formatString array be const; before the characters were, but the array was a modifiable global variable. (JSC::dateProtoFuncToLocaleString): Changed to call the new unified version of formatLocaleDate and remove the ifdef. (JSC::dateProtoFuncToLocaleDateString): Ditto. (JSC::dateProtoFuncToLocaleTimeString): Ditto.

• runtime/JSNotAnObject.cpp: (JSC::JSNotAnObject::toObject): Use the new ASSERT_UNUSED instead of the old UNUSED_PARAM.

• runtime/RegExp.cpp: (JSC::RegExp::RegExp): Changed to only use UNUSED_PARAM when the parameter is actually unused.

• wtf/TCSystemAlloc.cpp: (TCMalloc_SystemRelease): Changed to only use UNUSED_PARAM when the parameter is actually unused. (TCMalloc_SystemCommit): Changed to omit the argument names instead of using UNUSED_PARAM.

File:

• trunk/JavaScriptCore/wtf/TCSystemAlloc.cpp (modified) (2 diffs)

trunk/JavaScriptCore/wtf/TCSystemAlloc.cpp

@@ -384 +384 @@
 void TCMalloc_SystemRelease(void* start, size_t length)
 {
-  UNUSED_PARAM(start);
-  UNUSED_PARAM(length);
 #if HAVE(MADV_DONTNEED)
   if (FLAGS_malloc_devmem_start) {
@@ -427 +425 @@
   return;
 #endif
+
+#if !HAVE(MADV_DONTNEED) && !HAVE(MMAP)
+  UNUSED_PARAM(start);
+  UNUSED_PARAM(length);
+#endif
 }
 
 #if HAVE(VIRTUALALLOC)
-void TCMalloc_SystemCommit(void* start, size_t length)
+void TCMalloc_SystemCommit(void*, size_t)
 {
-    UNUSED_PARAM(start);
-    UNUSED_PARAM(length);
-}
-#endif
+}
+#endif