[Piglit] [V2 PATCH 12/12] summary: rename NewSummary to Summary

Dylan Baker baker.dylan.c at gmail.com
Fri Jun 28 06:49:29 PDT 2013


Since the old summary code is gone, the NewSummary class can now be
named just Summary. This should be clearer and easier to understand.
---
 framework/summary.py   | 4 ++--
 piglit-summary-html.py | 2 +-
 piglit-summary.py      | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/framework/summary.py b/framework/summary.py
index 9f1a10a..2eb3226 100644
--- a/framework/summary.py
+++ b/framework/summary.py
@@ -30,7 +30,7 @@ from mako.template import Template
 import core
 
 __all__ = [
-    'NewSummary',
+    'Summary',
 ]
 
 
@@ -242,7 +242,7 @@ class HTMLIndex(list):
                      'text': text})
 
 
-class NewSummary:
+class Summary:
     """
     This Summary class creates an initial object containing lists of tests
     including all, changes, problems, skips, regressions, and fixes. It then
diff --git a/piglit-summary-html.py b/piglit-summary-html.py
index 65e6653..276b187 100755
--- a/piglit-summary-html.py
+++ b/piglit-summary-html.py
@@ -92,7 +92,7 @@ def main():
         args.resultsFiles.extend(parse_listfile(args.list))
 
     # Create the HTML output
-    output = summary.NewSummary(args.resultsFiles)
+    output = summary.Summary(args.resultsFiles)
     output.generateHTML(args.summaryDir, args.exclude_details)
 
 
diff --git a/piglit-summary.py b/piglit-summary.py
index f07d644..99497ef 100755
--- a/piglit-summary.py
+++ b/piglit-summary.py
@@ -81,7 +81,7 @@ def main():
         args.results.extend(parse_listfile(args.list))
 
     # Generate the output
-    output = summary.NewSummary(args.results)
+    output = summary.Summary(args.results)
     output.generateText(args.diff, args.summary)
 
 
-- 
1.8.1.4



More information about the Piglit mailing list