RESOLVED FIXED 85110
[chromium] Fix compiler warning in CCSchedulerTest.cpp with gcc 4.6.3 
https://bugs.webkit.org/show_bug.cgi?id=85110
Summary [chromium] Fix compiler warning in CCSchedulerTest.cpp with gcc 4.6.3
Tony Chang
Reported 2012-04-27 16:41:22 PDT
[chromium] Fix compiler warning in CCSchedulerTest.cpp with gcc 4.6.3
Attachments
Patch (1.80 KB, patch)
2012-04-27 16:42 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tony Chang
Comment 1 2012-04-27 16:42:30 PDT
Created attachment 139306 [details] Patch
James Robinson
Comment 2 2012-04-27 16:50:37 PDT
Comment on attachment 139306 [details] Patch Can we enable this warning on other compilers/platforms?
WebKit Review Bot
Comment 3 2012-04-27 19:28:41 PDT
Comment on attachment 139306 [details] Patch Clearing flags on attachment: 139306 Committed r115527: <http://trac.webkit.org/changeset/115527>
WebKit Review Bot
Comment 4 2012-04-27 19:28:45 PDT
All reviewed patches have been landed. Closing bug.
Tony Chang
Comment 5 2012-05-02 15:08:33 PDT
It looks like this was added to clang here: http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20120312/054984.html Not sure if it's part of -Wall or not, maybe Nico knows?
Nico Weber
Comment 6 2012-05-02 15:10:16 PDT
It's part of -Wall, but clang suppresses the warning in a few cases (one of them got changed last week, so maybe clang will warn on this next week)
Note You need to log in before you can comment on or make changes to this bug.