Some Include What You Use suggestions were used too.
I checked this patch on my own build on RHEL 6. Regressions were OK.
Paths
| Differential D18810
[Clang-tidy] Fix readability-static-definition-in-anonymous-namespace warnings; other minor fixes. AbandonedPublic Authored by Eugene.Zelenko on Apr 5 2016, 5:20 PM.
Details
Summary Some Include What You Use suggestions were used too. I checked this patch on my own build on RHEL 6. Regressions were OK.
Diff Detail Event TimelineEugene.Zelenko retitled this revision from to [Clang-tidy] Fix readability-static-definition-in-anonymous-namespace warnings; other minor fixes.. Eugene.Zelenko updated this object. Comment Actions Please regenerate the patch with the full context.
This revision now requires changes to proceed.Apr 7 2016, 6:43 AM Comment Actions Actually, removal of static is not the right thing for LLVM:
http://llvm.org/docs/CodingStandards.html#anonymous-namespaces
Revision Contents
Diff 52748 clang-tidy/ClangTidy.cpp
clang-tidy/cppcoreguidelines/ProTypeMemberInitCheck.cpp
clang-tidy/misc/DanglingHandleCheck.cpp
clang-tidy/performance/FasterStringFindCheck.cpp
|
Did you make these changes automatically or use IWYU or a similar tool?