diff options
author | Lars Wirzenius <liw@liw.fi> | 2012-01-17 19:17:22 +0000 |
---|---|---|
committer | Lars Wirzenius <liw@liw.fi> | 2012-01-17 19:17:22 +0000 |
commit | d00e022642237ad9afffb623428153efd66e1e0a (patch) | |
tree | 4e435663d49f91789257f453bbded043636f989d | |
parent | cdbf176c8e6085b3d6cd2f45c76812e91eee1a61 (diff) | |
download | extrautils-d00e022642237ad9afffb623428153efd66e1e0a.tar.gz |
benchmark-cmd --cleanup-once now runs cleanups at the end
Also, a bit clearer code.
-rwxr-xr-x | benchmark-cmd | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/benchmark-cmd b/benchmark-cmd index d904602..0a50654 100755 --- a/benchmark-cmd +++ b/benchmark-cmd @@ -92,18 +92,21 @@ class Benchmarker(cliapp.Application): results.add_column('elapsed', '(s)', '%.1f') results.add_column('cmd', '', '%-0s', left=True) - first = True - for cmd in self.settings['command']: - if first or not self.settings['setup-once']: + commands = self.settings['command'] + if commands and self.settings['setup-once']: + self.setup() + for cmd in commands: + if not self.settings['setup-once']: self.setup() numbers = self.measure_cmd(cmd) self.verify() results.add_row(numbers + (cmd,)) - if first or not self.settings['cleanup-once']: + if not self.settings['cleanup-once']: self.cleanup() if self.settings['verbose']: self.output.write('\n') - first = False + if commands and self.settings['cleanup-once']: + self.cleanup() results.write_plaintext(self.output) |