Using splat with a dictionary makes sense when the dictionary is already available (like with locals()), but it isn't very clear.
Signed-off-by: Dylan Baker <[email protected]> --- framework/log.py | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/framework/log.py b/framework/log.py index 97afeb1..c967028 100644 --- a/framework/log.py +++ b/framework/log.py @@ -95,11 +95,12 @@ class Log(object): def __print(self, name, result): """ Do the actual printing """ - self._write_output(self.__output.format(**{'percent': self._percent(), - 'running': self._running(), - 'summary': self._summary(), - 'name': name, - 'result': result})) + self._write_output(self.__output.format( + percent=self._percent(), + running=self._running(), + summary=self._summary(), + name=name, + result=result)) @synchronized_self def log(self, name, result, value): @@ -147,5 +148,6 @@ class Log(object): return x def summary(self): - self._write_output(self.__summary_output.format(**{'percent': self._percent(), - 'summary': self._summary()})) + self._write_output(self.__summary_output.format( + percent=self._percent(), + summary=self._summary())) -- 2.0.0 _______________________________________________ Piglit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/piglit
