Merge pull request nuxeo#119 from sjbrown/minor-fixes

bdelbosc · bdelbosc · commit 14b712203264 · 2014-03-10T23:03:08.000+01:00
Minor fixes
diff --git a/src/funkload/ReportRenderHtmlGnuPlot.py b/src/funkload/ReportRenderHtmlGnuPlot.py
@@ -385,9 +385,9 @@ def createRPSTimeChart(self):
             "FF0000",
             "FF8C00",
                    ];
-        i = 0
-        last_cycle = self.cycles[-1]
-        for cycle in self.cycles:
+        for i, cycle in enumerate(self.cycles):
+            if i != 0:
+                plot_line += ', \\\n'
             dpath = gnuplot_scriptpath(self.report_dir,
                                        'time_rps-{0}.data'.format(cycle))
             #lines.append('set size 1,1\n')
@@ -402,9 +402,6 @@ def createRPSTimeChart(self):
             plot_line += ' w linespoints lw 1 lt 2 lc ' + \
                          'rgbcolor "#{0}" t "{1} CUs"'.format(colors[i % len(colors)],
                                                                      stats[cycle]['response'].cvus)
-            if cycle != last_cycle:
-                plot_line += ', \\\n'
-            i += 1
         lines.append(plot_line)
         #lines.append('unset multiplot\n')
         lines = self.fixXLabels('\n'.join(lines) + '\n')
diff --git a/src/funkload/ReportRenderOrg.py b/src/funkload/ReportRenderOrg.py
@@ -77,7 +77,7 @@ def org_footer(self):
 
 
 class RenderOrg(RenderRst):
-    """Render stats in ReST format."""
+    """Render stats in emacs org-mode format."""
     # number of slowest requests to display
     slowest_items = 5
     with_chart = True
