diff options
| author | Cassidy Burden <cburden@google.com> | 2016-08-01 12:10:35 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2016-08-01 15:56:10 -0700 |
| commit | a742f71e420a07c474a8fb9307426ddb0574dcf3 (patch) | |
| tree | 584a02ad1ab2c08f3efe2bca2efc7b022c2e45cd /binary_search_tool/binary_search_perforce.py | |
| parent | 660e682dae1e34076cb758694be67edfc7bc6ed4 (diff) | |
| download | toolchain-utils-a742f71e420a07c474a8fb9307426ddb0574dcf3.tar.gz | |
binary search tool: Change skip exit status to 125
Because exit status 2 means syntax error, it is confusing to also use
exit status 2 to mean "skip image". Change uses of status 2 to status
125.
Change-Id: I3620e3eb6e52523dc51804076284abf1e023d14f
Reviewed-on: https://chrome-internal-review.googlesource.com/272241
Commit-Ready: Cassidy Burden <cburden@google.com>
Tested-by: Cassidy Burden <cburden@google.com>
Reviewed-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'binary_search_tool/binary_search_perforce.py')
| -rwxr-xr-x | binary_search_tool/binary_search_perforce.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/binary_search_tool/binary_search_perforce.py b/binary_search_tool/binary_search_perforce.py index 2ff60467..3c1e9b51 100755 --- a/binary_search_tool/binary_search_perforce.py +++ b/binary_search_tool/binary_search_perforce.py @@ -94,13 +94,13 @@ class BinarySearcher(object): self.current, status)) self.logger.LogOutput(message, print_to_console=verbose) - assert status == 0 or status == 1 or status == 2 + assert status == 0 or status == 1 or status == 125 self.index_log.append(self.current) self.status_log.append(status) bsp = BinarySearchPoint(self.sorted_list[self.current], status, tag) self.points[self.current] = bsp - if status == 2: + if status == 125: self.skipped_indices.append(self.current) if status == 0 or status == 1: |