Thu, 15 Jan 2015 21:03:48 +0100
Integrate friendly tips from Tor colleagues to make (or not) 4.5 alpha 3;
This includes removal of overloaded (but unused) methods, and addition of
a overlooked call to DataStruct::SetData(nsISupports, uint32_t, bool.)
michael@0 | 1 | In file included from overridden_methods.cpp:5: |
michael@0 | 2 | ./overridden_methods.h:43:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 3 | virtual void SomeMethod(); |
michael@0 | 4 | ^ |
michael@0 | 5 | ./overridden_methods.h:47:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 6 | virtual void SomeInlineMethod() {} |
michael@0 | 7 | ^ |
michael@0 | 8 | ./overridden_methods.h:51:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 9 | virtual void SomeNonPureBaseMethod() {} |
michael@0 | 10 | ^ |
michael@0 | 11 | overridden_methods.cpp:24:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 12 | virtual void SomeMethod(); |
michael@0 | 13 | ^ |
michael@0 | 14 | overridden_methods.cpp:28:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 15 | virtual void SomeInlineMethod() {} |
michael@0 | 16 | ^ |
michael@0 | 17 | overridden_methods.cpp:32:11: warning: [chromium-style] Overriding method must be marked with OVERRIDE. |
michael@0 | 18 | virtual void SomeNonPureBaseMethod() {} |
michael@0 | 19 | ^ |
michael@0 | 20 | 6 warnings generated. |