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)
commitc839ca0ef111e8a7d1c2a894b7a336fd5c201c73
treeebe80599450e3f679d252b010c777549d37e5c92
parenta2b69e2aa860c725a90cfe80eb28a0c5851abe1e
JavaScriptCore: Revert r62689.
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