JavaScriptCore: Revert r62689.
authordumi@chromium.org <dumi@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jul 2010 22:20:40 +0000 (22:20 +0000)
committerdumi@chromium.org <dumi@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 7 Jul 2010 22:20:40 +0000 (22:20 +0000)
https://bugs.webkit.org/show_bug.cgi?id=41804

Reviewed by Adam Roben.

* runtime/Collector.cpp:
(JSC::Heap::freeBlocks):

WebKitLibraries: Disable MSVC warning 4288.
https://bugs.webkit.org/show_bug.cgi?id=41804

Reviewed by Adam Roben.

MSVC has a non-standard extension that allows variables declared
in for-loops to remain visible in the same scope even after
exiting the for-loop
(http://msdn.microsoft.com/en-us/library/bk5hc10s.aspx). The /Ze
option (turned on by default) enables all MSVC extensions, and
/Zc:forScope- tells the compiler to issue a C4288 warning when the
same variable is declared in the for-loop and re-declared later in
the same scope.

There seems to be a bug in VS2005 that erroneously enables
/Zc:forScope- even when that option is not specified
(http://connect.microsoft.com/VisualStudio/feedback/details/338010/bogus-compiler-warning-c4288). Looks
like our build got hit by that bug, so we need to disable warning
4288 to fix it.

* win/tools/vsprops/common.vsprops:

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

JavaScriptCore/ChangeLog
JavaScriptCore/runtime/Collector.cpp
WebKitLibraries/ChangeLog
WebKitLibraries/win/tools/vsprops/common.vsprops

index 6c33b38..02400d1 100644 (file)
@@ -1,3 +1,13 @@
+2010-07-07  Dumitru Daniliuc  <dumi@chromium.org>
+
+        Reviewed by Adam Roben.
+
+        Revert r62689.
+        https://bugs.webkit.org/show_bug.cgi?id=41804
+
+        * runtime/Collector.cpp:
+        (JSC::Heap::freeBlocks):
+
 2010-07-07  Adam Barth  <abarth@webkit.org>
 
         Reviewed by Sam Weinig.
index 71441dd..38f3ce5 100644 (file)
@@ -315,13 +315,10 @@ void Heap::freeBlocks()
 
     m_heap.nextCell = 0;
     m_heap.nextBlock = 0;
-
-    {
-        DeadObjectIterator it(m_heap, m_heap.nextBlock, m_heap.nextCell);
-        DeadObjectIterator end(m_heap, m_heap.usedBlocks);
-        for ( ; it != end; ++it)
-            (*it)->~JSCell();
-    }
+    DeadObjectIterator it(m_heap, m_heap.nextBlock, m_heap.nextCell);
+    DeadObjectIterator end(m_heap, m_heap.usedBlocks);
+    for ( ; it != end; ++it)
+        (*it)->~JSCell();
 
     ASSERT(!protectedObjectCount());
 
index 68cd1c4..f791ac2 100644 (file)
@@ -1,3 +1,27 @@
+2010-07-07  Dumitru Daniliuc  <dumi@chromium.org>
+
+        Reviewed by Adam Roben.
+
+        Disable MSVC warning 4288.
+        https://bugs.webkit.org/show_bug.cgi?id=41804
+
+        MSVC has a non-standard extension that allows variables declared
+        in for-loops to remain visible in the same scope even after
+        exiting the for-loop
+        (http://msdn.microsoft.com/en-us/library/bk5hc10s.aspx). The /Ze
+        option (turned on by default) enables all MSVC extensions, and
+        /Zc:forScope- tells the compiler to issue a C4288 warning when the
+        same variable is declared in the for-loop and re-declared later in
+        the same scope.
+
+        There seems to be a bug in VS2005 that erroneously enables
+        /Zc:forScope- even when that option is not specified
+        (http://connect.microsoft.com/VisualStudio/feedback/details/338010/bogus-compiler-warning-c4288). Looks
+        like our build got hit by that bug, so we need to disable warning
+        4288 to fix it.
+
+        * win/tools/vsprops/common.vsprops:
+
 2010-07-01  Simon Fraser  <simon.fraser@apple.com>
 
         Reviewed by Sam Weinig.
index 28dfb67..b9c710c 100644 (file)
@@ -15,7 +15,7 @@
                WarningLevel="4"
                WarnAsError="true"
                DebugInformationFormat="3"
-               DisableSpecificWarnings="4018;4068;4099;4100;4127;4138;4180;4189;4201;4244;4251;4275;4291;4305;4344;4355;4389;4503;4505;4510;4512;4610;4706;4800;4951;4952;4996;6011;6031;6211;6246;6255;6387"
+               DisableSpecificWarnings="4018;4068;4099;4100;4127;4138;4180;4189;4201;4244;4251;4275;4288;4291;4305;4344;4355;4389;4503;4505;4510;4512;4610;4706;4800;4951;4952;4996;6011;6031;6211;6246;6255;6387"
        />
        <Tool
                Name="VCLibrarianTool"