The find-all-symbols tool generates a yaml symbol database for
include-fixer.
The symbol matcher is originally written by Xiaoyi Liu.
Paths
| Differential D19482
[include-fixer] Add a find-all-symbols tool for include-fixer. ClosedPublic Authored by hokein on Apr 25 2016, 8:14 AM.
Details Summary The find-all-symbols tool generates a yaml symbol database for The symbol matcher is originally written by Xiaoyi Liu.
Diff Detail Event TimelineComment Actions First, please run this through clang-format with LLVM style. This aligns all the & and * to the name instead of the type. There are also some underscore_names left that need conversion to PascalCase.
hokein marked 3 inline comments as done. hokein edited edge metadata. Comment ActionsAddress review comments. This revision is now accepted and ready to land.Apr 26 2016, 7:55 AM hokein marked 2 inline comments as done. hokein edited edge metadata. Comment ActionsUpdate.
hokein marked 13 inline comments as done. Comment ActionsAdd Daniel's comments.
djasper edited edge metadata. Comment ActionsLooks good.
Closed by commit rL267719: [include-fixer] Add a find-all-symbols tool for include-fixer. (authored by hokein). ยท Explain WhyApr 27 2016, 7:33 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 54856 include-fixer/CMakeLists.txt
include-fixer/find-all-symbols/CMakeLists.txt
include-fixer/find-all-symbols/FindAllSymbols.h
include-fixer/find-all-symbols/FindAllSymbols.cpp
include-fixer/find-all-symbols/SymbolInfo.h
include-fixer/find-all-symbols/SymbolInfo.cpp
include-fixer/find-all-symbols/tool/CMakeLists.txt
include-fixer/find-all-symbols/tool/FindAllSymbolsMain.cpp
include-fixer/find-all-symbols/tool/run-find-all-symbols.py
unittests/include-fixer/CMakeLists.txt
unittests/include-fixer/find-all-symbols/CMakeLists.txt
unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp
|
nit: use = default instead of {}