diff options
| author | Caroline Tice <cmtice@google.com> | 2017-04-06 17:16:05 -0700 |
|---|---|---|
| committer | chrome-bot <chrome-bot@chromium.org> | 2017-04-08 19:40:20 -0700 |
| commit | f6ef4395fe1896ba68c80e52cb24763b0fcfe7f8 (patch) | |
| tree | 7612123711db98f0746e56b66368d16b388192bd /crosperf/experiment_file.py | |
| parent | 8c18be1425c8a4ecfc059a7c637fc93f33edab1f (diff) | |
| download | toolchain-utils-f6ef4395fe1896ba68c80e52cb24763b0fcfe7f8.tar.gz | |
[toolchain-utils] Fix remaining lint errors in toolchain-utils.
In addition to fixing the lint errors, this also fixes the Python
formatting issues (ran tc_pyformat on nearly all the files).
BUG=chromium:570450
TEST=Ran all crosperf & bisect tool unit tests.
Ran afe_lock_machine.py (check machine status)
Ran full crosperf test (octane, speedometer, BootPerf) on alex.
Change-Id: Ic86f9192801ac67769f3de30f1c5f0d203ce0831
Reviewed-on: https://chromium-review.googlesource.com/471886
Commit-Ready: Caroline Tice <cmtice@chromium.org>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Diffstat (limited to 'crosperf/experiment_file.py')
| -rw-r--r-- | crosperf/experiment_file.py | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/crosperf/experiment_file.py b/crosperf/experiment_file.py index 016e9d86..57eb52dc 100644 --- a/crosperf/experiment_file.py +++ b/crosperf/experiment_file.py @@ -114,8 +114,8 @@ class ExperimentFile(object): elif ExperimentFile._OPEN_SETTINGS_RE.match(line): new_settings = self._ParseSettings(reader) if new_settings.name in settings_names: - raise SyntaxError("Duplicate settings name: '%s'." % - new_settings.name) + raise SyntaxError( + "Duplicate settings name: '%s'." % new_settings.name) settings_names[new_settings.name] = True self.all_settings.append(new_settings) elif ExperimentFile._FIELD_VALUE_RE.match(line): @@ -160,11 +160,8 @@ class ExperimentFile(object): autotest_path = '' if autotest_field.assigned: autotest_path = autotest_field.GetString() - image_path, autotest_path = settings.GetXbuddyPath(value, - autotest_path, - board, - chromeos_root, - 'quiet') + image_path, autotest_path = settings.GetXbuddyPath( + value, autotest_path, board, chromeos_root, 'quiet') res += '\t#actual_image: %s\n' % image_path if not autotest_field.assigned: res += '\t#actual_autotest_path: %s\n' % autotest_path |