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

Unified Diff: tools/clang/scripts/run_tool.py

Issue 761903003: Update from https://crrev.com/306655 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « tools/android/adb_remote_setup.sh ('k') | tools/git/move_source_file.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/clang/scripts/run_tool.py
diff --git a/tools/clang/scripts/run_tool.py b/tools/clang/scripts/run_tool.py
index 3725ca3e97719101e68dc285c9e72bdf69b6b65e..58a9a86570c6b3b203d5a0004ce56d04426db314 100755
--- a/tools/clang/scripts/run_tool.py
+++ b/tools/clang/scripts/run_tool.py
@@ -314,14 +314,14 @@ def main(argv):
clang_format_diff_path = None
if len(argv) == 3 and argv[2] == '--all':
- filenames = frozenset(_GetFilesFromCompileDB(argv[1]))
+ filenames = set(_GetFilesFromCompileDB(argv[1]))
else:
- filenames = frozenset(_GetFilesFromGit(argv[2:]))
- # Filter out files that aren't C/C++/Obj-C/Obj-C++.
- extensions = frozenset(('.c', '.cc', '.m', '.mm'))
- dispatcher = _CompilerDispatcher(argv[0], argv[1],
- [f for f in filenames
- if os.path.splitext(f)[1] in extensions])
+ filenames = set(_GetFilesFromGit(argv[2:]))
+ # Filter out files that aren't C/C++/Obj-C/Obj-C++.
+ extensions = frozenset(('.c', '.cc', '.m', '.mm'))
+ filenames = [f for f in filenames
+ if os.path.splitext(f)[1] in extensions]
+ dispatcher = _CompilerDispatcher(argv[0], argv[1], filenames)
dispatcher.Run()
# Filter out edits to files that aren't in the git repository, since it's not
# useful to modify files that aren't under source control--typically, these
« no previous file with comments | « tools/android/adb_remote_setup.sh ('k') | tools/git/move_source_file.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698