[llvm-commits] [zorg] r130363 - /zorg/trunk/lnt/lnt/util/NTEmailReport.py

[Daniel Dunbar]

Author: ddunbar
Date: Wed Apr 27 19:55:22 2011
New Revision: 130363

URL: http://llvm.org/viewvc/llvm-project?rev=130363&view=rev
Log:
LNT: Fix a variable scoping issue which generated weird reports when used with
multiple parameter sets.

Modified:
    zorg/trunk/lnt/lnt/util/NTEmailReport.py

Modified: zorg/trunk/lnt/lnt/util/NTEmailReport.py
URL: http://llvm.org/viewvc/llvm-project/zorg/trunk/lnt/lnt/util/NTEmailReport.py?rev=130363&r1=130362&r2=130363&view=diff
==============================================================================
--- zorg/trunk/lnt/lnt/util/NTEmailReport.py (original)
+++ zorg/trunk/lnt/lnt/util/NTEmailReport.py Wed Apr 27 19:55:22 2011
@@ -307,8 +307,8 @@
         print >>html_report, """
 <p>
 <h3>Changes Detail</h3>"""
-        for name,items,show_perf in items_info:
-            if not items or name == 'Unchanged Tests':
+        for test_name,items,show_perf in items_info:
+            if not items or test_name == 'Unchanged Tests':
                 continue
 
             show_pset = items.items()[0][0] or len(items) > 1
@@ -316,8 +316,8 @@
                 (pset, 'pset.%d' % i)
                 for i,pset in enumerate(ts_summary.parameter_sets))
             print >>report
-            print >>report, name
-            print >>report, '-' * len(name)
+            print >>report, test_name
+            print >>report, '-' * len(test_name)
             for pset,tests in items.items():
                 if show_perf:
                     tests.sort(key = lambda (_,cr): -abs(cr.pct_delta))
@@ -332,7 +332,6 @@
                     (get_last_component(t), t)
                     for t in tests)
 
-                test_name = name
                 for group,grouped_tests in Util.sorted(grouped.items()):
                     group_name = {
                         "" : "(ungrouped)",