Changeset 50171 in webkit

Timestamp:

Oct 27, 2009 2:04:20 PM (15 years ago)

Author:

eric@webkit.org

Message:

2009-10-27 Carol Szabo <​carol.szabo@nokia.com>

Reviewed by Darin Adler.

Give an ability to WebKit to free statically allocated pointers
before quit.
​https://bugs.webkit.org/show_bug.cgi?id=27980

No new tests for this as the change does not affect WebKit
functionality, but only memory usage.

• JavaScriptCore/wtf/StdLibExtras.h: Allowed for the override of DEFINE_STATIC_LOCAL via compiler commandline options or elsewhere in the code. Useful for implementing custom builds that free resources allocated via DEFINE_STATIC_LOCAL.

Location:

trunk

Files:

• JavaScriptCore/wtf/StdLibExtras.h (modified) (2 diffs)
• WebCore/ChangeLog (modified) (1 diff)

trunk/JavaScriptCore/wtf/StdLibExtras.h

@@ -33 +33 @@
 //  it is leaked so that its destructors are not called at exit. Using this
 //  macro also allows workarounds a compiler bug present in Apple's version of GCC 4.0.1.
+#ifndef DEFINE_STATIC_LOCAL
 #if COMPILER(GCC) && defined(__APPLE_CC__) && __GNUC__ == 4 && __GNUC_MINOR__ == 0 && __GNUC_PATCHLEVEL__ == 1
 #define DEFINE_STATIC_LOCAL(type, name, arguments) \
@@ -40 +41 @@
 #define DEFINE_STATIC_LOCAL(type, name, arguments) \
     static type& name = *new type arguments
+#endif
 #endif
 

trunk/WebCore/ChangeLog

@@ +1 @@
+2009-10-27  Carol Szabo  <carol.szabo@nokia.com>
+
+        Reviewed by Darin Adler.
+
+        Give an ability to WebKit to free statically allocated pointers
+        before quit.
+        https://bugs.webkit.org/show_bug.cgi?id=27980
+
+        No new tests for this as the change does not affect WebKit
+        functionality, but only memory usage.
+
+        * JavaScriptCore/wtf/StdLibExtras.h:
+        Allowed for the override of DEFINE_STATIC_LOCAL via compiler 
+        commandline options or elsewhere in the code. Useful for 
+        implementing custom builds that free resources allocated via
+        DEFINE_STATIC_LOCAL.
+
 2009-10-27  Dumitru Daniliuc  <dumi@chromium.org>