RESOLVED FIXED 68654
We should add support for OVERRIDE and FINAL annotations 
https://bugs.webkit.org/show_bug.cgi?id=68654
Summary We should add support for OVERRIDE and FINAL annotations
James Robinson
Reported 2011-09-22 13:45:18 PDT
These C++11 annotations are useful for detecting programming errors. We've gained a lot of utility from these in the Chromium repo, and I've locally written plenty of bugs that these would catch. MSVS2005, recent clangs, and GCC 4.7+ support these annotations. We should be able to define these annotations as no-ops on other compilers and still get broad coverage from developers and across the buildbots.
Attachments
Patch (1.58 KB, patch)
2011-09-22 13:55 PDT, Anders Carlsson 		no flags
DetailsFormatted DiffDiff
Patch (1.63 KB, patch)
2011-09-22 14:08 PDT, Anders Carlsson 		hyatt: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Anders Carlsson
Comment 1 2011-09-22 13:55:42 PDT
Created attachment 108396 [details] Patch
Anders Carlsson
Comment 2 2011-09-22 14:08:48 PDT
Created attachment 108400 [details] Patch
Dave Hyatt
Comment 3 2011-09-22 15:31:37 PDT
Comment on attachment 108400 [details] Patch r=me
Darin Adler
Comment 4 2011-09-22 16:08:10 PDT
Can’t wait!
Anders Carlsson
Comment 5 2011-09-23 09:48:05 PDT
Committed r95823: <http://trac.webkit.org/changeset/95823>
Note You need to log in before you can comment on or make changes to this bug.