[LNT] r237655 - Remove getter for samples, it is not used

Chris Matthews cmatthews5 at apple.com
Mon May 18 18:59:12 PDT 2015


Author: cmatthews
Date: Mon May 18 20:59:12 2015
New Revision: 237655

URL: http://llvm.org/viewvc/llvm-project?rev=237655&view=rev
Log:
Remove getter for samples, it is not used

It could be a property anyways.

Modified:
    lnt/trunk/lnt/server/reporting/analysis.py
    lnt/trunk/lnt/server/ui/templates/v4_run.html

Modified: lnt/trunk/lnt/server/reporting/analysis.py
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/reporting/analysis.py?rev=237655&r1=237654&r2=237655&view=diff
==============================================================================
--- lnt/trunk/lnt/server/reporting/analysis.py (original)
+++ lnt/trunk/lnt/server/reporting/analysis.py Mon May 18 20:59:12 2015
@@ -45,8 +45,6 @@ class ComparisonResult:
         self.confidence_lv = confidence_lv
         self.bigger_is_better = bigger_is_better
 
-    def get_samples(self):
-        return self.samples
 
     def is_result_interesting(self):
         """is_result_interesting() -> bool

Modified: lnt/trunk/lnt/server/ui/templates/v4_run.html
URL: http://llvm.org/viewvc/llvm-project/lnt/trunk/lnt/server/ui/templates/v4_run.html?rev=237655&r1=237654&r2=237655&view=diff
==============================================================================
--- lnt/trunk/lnt/server/ui/templates/v4_run.html (original)
+++ lnt/trunk/lnt/server/ui/templates/v4_run.html Mon May 18 20:59:12 2015
@@ -70,7 +70,7 @@
   {% if options.show_all_samples %}
     <td>
       [
-      {%- for v in cr.get_samples() -%}
+      {%- for v in cr.samples -%}
         {{ ", " if not loop.first else "" }}
         {{ "%.4f" % v }}
       {%- endfor -%}
@@ -79,7 +79,7 @@
   {% endif %}
 
   {% if options.show_sample_counts %}
-    <td>{{cr.get_samples()|length}}</td>
+    <td>{{cr.samples|length}}</td>
   {% endif %}
 
 {% endmacro %}





More information about the llvm-commits mailing list