| OLD | NEW |
|---|
| (Empty) | |
| 1 In file included from overridden_methods.cpp:5: |
| 2 ./overridden_methods.h:25:29: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 3 virtual void SomeMethod() = 0; |
| 4 ^ |
| 5 override |
| 6 ./overridden_methods.h:38:42: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 7 virtual void WebKitModifiedSomething() {}; |
| 8 ^ |
| 9 override |
| 10 ./overridden_methods.h:46:27: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 11 virtual ~DerivedClass() {} |
| 12 ^ |
| 13 override |
| 14 ./overridden_methods.h:48:28: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 15 virtual void SomeMethod(); |
| 16 ^ |
| 17 override |
| 18 ./overridden_methods.h:52:35: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 19 virtual void SomeInlineMethod() {} |
| 20 ^ |
| 21 override |
| 22 ./overridden_methods.h:54:41: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 23 virtual void WebKitModifiedSomething(); |
| 24 ^ |
| 25 override |
| 26 ./overridden_methods.h:56:40: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 27 virtual void SomeConstMethod() const {} |
| 28 ^ |
| 29 override |
| 30 ./overridden_methods.h:58:54: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 31 virtual void SomeMethodWithExceptionSpec() throw() {} |
| 32 ^ |
| 33 override |
| 34 ./overridden_methods.h:61:68: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 35 virtual void SomeConstMethodWithExceptionSpec() const throw(int) {} |
| 36 ^ |
| 37 override |
| 38 ./overridden_methods.h:63:40: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 39 virtual void SomeNonPureBaseMethod() {} |
| 40 ^ |
| 41 override |
| 42 ./overridden_methods.h:65:39: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 43 virtual void SomeMethodWithComment(); // This is a comment. |
| 44 ^ |
| 45 override |
| 46 ./overridden_methods.h:67:47: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 47 virtual void SomeMethodWithCommentAndBody() {} // This is a comment. |
| 48 ^ |
| 49 override |
| 50 overridden_methods.cpp:15:29: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 51 virtual void SomeMethod() = 0; |
| 52 ^ |
| 53 override |
| 54 overridden_methods.cpp:22:41: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 55 virtual ~ImplementationDerivedClass() {} |
| 56 ^ |
| 57 override |
| 58 overridden_methods.cpp:24:28: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 59 virtual void SomeMethod(); |
| 60 ^ |
| 61 override |
| 62 overridden_methods.cpp:28:35: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 63 virtual void SomeInlineMethod() {} |
| 64 ^ |
| 65 override |
| 66 overridden_methods.cpp:30:41: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 67 virtual void WebKitModifiedSomething(); |
| 68 ^ |
| 69 override |
| 70 overridden_methods.cpp:32:40: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 71 virtual void SomeConstMethod() const {} |
| 72 ^ |
| 73 override |
| 74 overridden_methods.cpp:34:54: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 75 virtual void SomeMethodWithExceptionSpec() throw() {} |
| 76 ^ |
| 77 override |
| 78 overridden_methods.cpp:37:68: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 79 virtual void SomeConstMethodWithExceptionSpec() const throw(int) {} |
| 80 ^ |
| 81 override |
| 82 overridden_methods.cpp:39:40: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 83 virtual void SomeNonPureBaseMethod() {} |
| 84 ^ |
| 85 override |
| 86 overridden_methods.cpp:41:39: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 87 virtual void SomeMethodWithComment(); // This is a comment. |
| 88 ^ |
| 89 override |
| 90 overridden_methods.cpp:43:47: warning: [chromium-style] Overriding method must b
e marked with 'override' or 'final'. |
| 91 virtual void SomeMethodWithCommentAndBody() {} // This is a comment. |
| 92 ^ |
| 93 override |
| 94 23 warnings generated. |
| OLD | NEW |
|---|
Chromium Code Reviews
