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

Side by Side Diff: docs/clang_tool_refactoring.md

Issue 2246263002: Skip parameter replacements where old text != old name. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Detecting buggy ParmVarDecls without looking at the source text. Created 4 years, 3 months 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 | « no previous file | tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Clang Tool Refactoring 1 # Clang Tool Refactoring
2 2
3 [TOC] 3 [TOC]
4 4
5 ## Introduction 5 ## Introduction
6 6
7 Clang tools can help with global refactorings of Chromium code. Clang tools can 7 Clang tools can help with global refactorings of Chromium code. Clang tools can
8 take advantage of clang's AST to perform refactorings that would be impossible 8 take advantage of clang's AST to perform refactorings that would be impossible
9 with a traditional find-and-replace regexp: 9 with a traditional find-and-replace regexp:
10 10
(...skipping 154 matching lines...) Expand 10 before | Expand all | Expand 10 after
165 stmt->dumpColor(); 165 stmt->dumpColor();
166 ``` 166 ```
167 167
168 By default, the script hides the output of the tool. The easiest way to change 168 By default, the script hides the output of the tool. The easiest way to change
169 that is to `return 1` from the `main()` function of the clang tool. 169 that is to `return 1` from the `main()` function of the clang tool.
170 170
171 ## Testing 171 ## Testing
172 Synposis: 172 Synposis:
173 173
174 ```shell 174 ```shell
175 test_tool.py <tool name> 175 tools/clang/scripts/test_tool.py <tool name>
176 ``` 176 ```
177 177
178 The name of the tool binary and the subdirectory for the tool in 178 The name of the tool binary and the subdirectory for the tool in
179 `//tools/clang` must match. The test runner finds all files that match the 179 `//tools/clang` must match. The test runner finds all files that match the
180 pattern `//tools/clang/<tool name>/tests/*-original.cc`, runs the tool across 180 pattern `//tools/clang/<tool name>/tests/*-original.cc`, runs the tool across
181 those files, and compared it to the `*-expected.cc` version. If there is a 181 those files, and compared it to the `*-expected.cc` version. If there is a
182 mismatch, the result is saved in `*-actual.cc`. 182 mismatch, the result is saved in `*-actual.cc`.
OLDNEW
« no previous file with comments | « no previous file | tools/clang/rewrite_to_chrome_style/RewriteToChromeStyle.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698