Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5)

Side by Side Diff: tools/clang/plugins/tests/weak_ptr_factory.txt

Issue 740083002: Clang plugin: Allow >1 WeakPtrFactory member in a class (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Some more documentation. Created 6 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « tools/clang/plugins/tests/weak_ptr_factory.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 weak_ptr_factory.cpp:35:38: warning: [chromium-style] WeakPtrFactory members whi ch refer to their outer class must be the last member in the outer class definit ion. 1 weak_ptr_factory.cpp:43:38: warning: [chromium-style] WeakPtrFactory members whi ch refer to their outer class must be the last member in the outer class definit ion.
2 base::WeakPtrFactory<FactoryFirst> factory_; 2 base::WeakPtrFactory<FactoryFirst> factory_;
3 ^ 3 ^
4 weak_ptr_factory.cpp:41:39: warning: [chromium-style] WeakPtrFactory members whi ch refer to their outer class must be the last member in the outer class definit ion. 4 weak_ptr_factory.cpp:49:39: warning: [chromium-style] WeakPtrFactory members whi ch refer to their outer class must be the last member in the outer class definit ion.
5 base::WeakPtrFactory<FactoryMiddle> factory_; 5 base::WeakPtrFactory<FactoryMiddle> factory_;
6 ^ 6 ^
7 2 warnings generated. 7 weak_ptr_factory.cpp:55:44: warning: [chromium-style] WeakPtrFactory members whi ch refer to their outer class must be the last member in the outer class definit ion.
8 base::WeakPtrFactory<TwoFactoriesOneBad> factory1_;
9 ^
10 3 warnings generated.
OLDNEW
« no previous file with comments | « tools/clang/plugins/tests/weak_ptr_factory.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698