<div dir="ltr">You're right, I'll change that.</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jul 2, 2013 at 11:39 AM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 06/28/2013 06:49 AM, Dylan Baker wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This information is actually quite useful. It should be noted the<br>
limitation of the text syntax, in that this is the fixes and regressions<br>
(and even changes) from the second to last to last provided json file.<br>
---<br>
  framework/summary.py | 22 ++++++++++++----------<br>
  1 file changed, 12 insertions(+), 10 deletions(-)<br>
<br>
diff --git a/framework/summary.py b/framework/summary.py<br>
index 7686b0e..305f5eb 100644<br>
--- a/framework/summary.py<br>
+++ b/framework/summary.py<br>
@@ -824,7 +824,7 @@ class NewSummary:<br>
<br>
          # If there are more than one set of results we need to find changes<br>
          if len(self.results) > 1:<br>
-            self.__generate_lists(['<u></u>changes'])<br>
+            self.__generate_lists(['<u></u>changes', 'fixes', 'regressions'])<br>
<br>
          # Print the name of the test and the status from each test run<br>
          if not summary:<br>
@@ -832,21 +832,23 @@ class NewSummary:<br>
                  for test in self.tests['changes']:<br>
                      print "%(test)s: %(statuses)s" % {'test': test, 'statuses':<br>
                      ' '.join([i.tests.get(test, {'result': 'skip'})['result']<br>
-                        for i in self.results])}<br>
+                             for i in self.results])}<br>
              else:<br>
                  for test in self.tests['all']:<br>
                      print "%(test)s: %(statuses)s" % {'test': test, 'statuses':<br>
                      ' '.join([i.tests.get(test, {'result': 'skip'})['result']<br>
-                        for i in self.results])}<br>
+                             for i in self.results])}<br>
</blockquote>
<br></div></div>
These two lines seem to be whitespace fixes that look like they belong in patch 6.  Not a big deal, but.<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
          # Print the summary<br>
          print "summary:"<br>
-        print "   pass: %d" % self.totals['pass']<br>
-        print "   fail: %d" % self.totals['fail']<br>
-        print "  crash: %d" % self.totals['crash']<br>
-        print "   skip: %d" % self.totals['skip']<br>
-        print "   warn: %d" % self.totals['warn']<br>
+        print "       pass: %d" % self.totals['pass']<br>
+        print "       fail: %d" % self.totals['fail']<br>
+        print "      crash: %d" % self.totals['crash']<br>
+        print "       skip: %d" % self.totals['skip']<br>
+        print "       warn: %d" % self.totals['warn']<br>
          if self.tests['changes']:<br>
-            print "changes: %d" % len(self.tests['changes'])<br>
+            print "    changes: %d" % len(self.tests['changes'])<br>
+            print "      fixes: %d" % len(self.tests['fixes'])<br>
+            print "regressions: %d" % len(self.tests['regressions'])<br>
<br>
-        print "  total: %d" % sum(self.totals.values())<br>
+        print "      total: %d" % sum(self.totals.values())<br>
<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>