diff options
author | cmtice <cmtice@google.com> | 2014-05-21 08:39:15 -0700 |
---|---|---|
committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2014-05-21 23:45:34 +0000 |
commit | ba0d0de24f34f008c150b9b5fab2f6dfa4c5217b (patch) | |
tree | 26672cfb5e943451a0633d15f2e5ad3f090c9d79 /crosperf | |
parent | 798a8fa986db930786b2a6777db0f3b06db995f6 (diff) | |
download | toolchain-utils-ba0d0de24f34f008c150b9b5fab2f6dfa4c5217b.tar.gz |
Fix bug, parameter name masked module name.
New 'logger' parameter name in __init__ function masked module
name. This CL renames the parameter to something else.
BUG=None
TEST=Ran crosperf with the fix and it worked.
Change-Id: I19b5e452d50ce9937662ec3ee36ebb79747985f5
Reviewed-on: https://chrome-internal-review.googlesource.com/164135
Reviewed-by: Luis Lozano <llozano@chromium.org>
Commit-Queue: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf')
-rw-r--r-- | crosperf/experiment_runner.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crosperf/experiment_runner.py b/crosperf/experiment_runner.py index cead9d37..3a2d600c 100644 --- a/crosperf/experiment_runner.py +++ b/crosperf/experiment_runner.py @@ -24,9 +24,9 @@ class ExperimentRunner(object): STATUS_TIME_DELAY = 30 THREAD_MONITOR_DELAY = 2 - def __init__(self, experiment, logger=None, cmd_exec=None): + def __init__(self, experiment, log=None, cmd_exec=None): self._experiment = experiment - self.l = logger or logger.GetLogger(experiment.log_dir) + self.l = log or logger.GetLogger(experiment.log_dir) self._ce = cmd_exec or command_executer.GetCommandExecuter(self.l) self._terminated = False if experiment.log_level != "verbose": |