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

Unified Diff: trychange.py

Issue 8133001: Remove |file_list| parameter from |TryChange()|. (Closed) Base URL: http://src.chromium.org/svn/trunk/tools/depot_tools/
Patch Set: '' Created 9 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « git_try.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trychange.py
===================================================================
--- trychange.py (revision 103729)
+++ trychange.py (working copy)
@@ -490,16 +490,19 @@
def TryChange(argv,
change,
- file_list,
swallow_exception,
prog=None,
extra_epilog=None):
"""
Args:
argv: Arguments and options.
- file_list: Default value to pass to --file.
+ change: Change instance corresponding to the CL.
swallow_exception: Whether we raise or swallow exceptions.
"""
+ file_list = []
+ if change:
+ file_list = [f.LocalPath() for f in change.AffectedFiles()]
+
# Parse argv
parser = optparse.OptionParser(usage=USAGE,
version=__version__,
@@ -806,4 +809,4 @@
if __name__ == "__main__":
fix_encoding.fix_encoding()
- sys.exit(TryChange(None, None, [], False))
+ sys.exit(TryChange(None, None, False))
« no previous file with comments | « git_try.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698