OLD | NEW |
---|---|
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "overridden_methods.h" | 5 #include "overridden_methods.h" |
6 | 6 |
7 // Fill in the implementations | 7 // Fill in the implementations |
8 void DerivedClass::SomeMethod() {} | 8 void DerivedClass::SomeMethod() {} |
9 void DerivedClass::SomeOtherMethod() {} | 9 void DerivedClass::SomeOtherMethod() {} |
10 void DerivedClass::WebKitModifiedSomething() {} | 10 void DerivedClass::WebKitModifiedSomething() {} |
11 | 11 |
12 class ImplementationInterimClass : public BaseClass { | |
13 public: | |
14 // Should not warn about pure virtual methods. | |
15 virtual void SomeMethod() = 0; | |
16 }; | |
17 | |
18 class ImplementationDerivedClass : public ImplementationInterimClass, | |
19 public webkit_glue::WebKitObserverImpl { | |
20 public: | |
21 // Should not warn about destructors. | |
22 virtual ~ImplementationDerivedClass() {} | |
23 // Should warn. | |
24 virtual void SomeMethod(); | |
25 // Should not warn if marked as override. | |
26 virtual void SomeOtherMethod() override; | |
27 // Should warn for inline implementations. | |
Nico
2012/04/17 00:46:01
in cpp files?
| |
28 virtual void SomeInlineMethod() {} | |
29 // Should not warn if overriding a method whose origin is WebKit. | |
30 virtual void WebKitModifiedSomething(); | |
31 // Should warn if overridden method isn't pure. | |
32 virtual void SomeNonPureBaseMethod() {} | |
33 }; | |
34 | |
12 int main() { | 35 int main() { |
13 DerivedClass something; | 36 DerivedClass something; |
37 ImplementationDerivedClass something_else; | |
14 } | 38 } |
OLD | NEW |