<p dir="ltr">I don't have a reasonable computer with me, if the pep8 till likes it you have my acked-by</p>
<div class="gmail_quote">On Jul 16, 2015 21:43, "Vinson Lee" <<a href="mailto:vlee@freedesktop.org">vlee@freedesktop.org</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Signed-off-by: Vinson Lee <<a href="mailto:vlee@freedesktop.org">vlee@freedesktop.org</a>><br>
---<br>
 framework/backends/json.py | 12 ++++++------<br>
 1 file changed, 6 insertions(+), 6 deletions(-)<br>
<br>
diff --git a/framework/backends/json.py b/framework/backends/json.py<br>
index b40eda7..b747e29 100644<br>
--- a/framework/backends/json.py<br>
+++ b/framework/backends/json.py<br>
@@ -186,7 +186,7 @@ def load_results(filename, compression_):<br>
         # Look for a compressed result first, then a bare result, finally for<br>
         # an old main file<br>
         for name in ['results.json.{}'.format(compression_),<br>
-                     'results.json',<br>
+                     'results.json',<br>
                      'main']:<br>
             if os.path.exists(os.path.join(filename, name)):<br>
                 filepath = os.path.join(filename, name)<br>
@@ -214,7 +214,7 @@ def _load(results_file):<br>
     """Load a json results instance and return a TestrunResult.<br>
<br>
     This function converts an existing, fully completed json run.<br>
-<br>
+<br>
     """<br>
     result = results.TestrunResult()<br>
     result.results_version = 0  # This should get overwritten<br>
@@ -317,7 +317,7 @@ def _update_results(results, filepath):<br>
 def _write(results, file_):<br>
     """WRite the values of the results out to a file."""<br>
     with write_compressed(file_) as f:<br>
-        json.dump({k:v for k, v in results.__dict__.iteritems()},<br>
+        json.dump({k: v for k, v in results.__dict__.iteritems()},<br>
                   f,<br>
                   default=piglit_encoder,<br>
                   indent=INDENT)<br>
@@ -355,8 +355,8 @@ def _update_zero_to_one(results):<br>
         #<br>
         # This expects that the order of info is rougly returncode, errors,<br>
         # output, *extra it can handle having extra information in the middle,<br>
-        if (None in [test.get('out'), test.get('err'), test.get('returncode')]<br>
-                and test.get('info')):<br>
+        if (None in [test.get('out'), test.get('err'),<br>
+                     test.get('returncode')] and test.get('info')):<br>
<br>
             # This attempts to split everything before Errors: as a returncode,<br>
             # and everything before Output: as Errors, and everything else as<br>
@@ -517,7 +517,7 @@ def _update_three_to_four(results):<br>
     return results<br>
<br>
<br>
-def _update_four_to_five(results):<br>
+def _update_four_to_five(results):<br>
     """Updates json results from version 4 to version 5."""<br>
     new_tests = {}<br>
<br>
--<br>
2.4.5<br>
<br>
_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</blockquote></div>