[tor-commits] [arm/master] Dropping GraphStats' is_visible() method

atagar at torproject.org atagar at torproject.org
Tue Oct 7 16:53:30 UTC 2014


commit e93fa0978d950586b1cace2320a07c37268c9ed4
Author: Damian Johnson <atagar at torproject.org>
Date:   Sun Oct 5 13:22:03 2014 -0700

    Dropping GraphStats' is_visible() method
    
    Method's completely unused, so easy one to drop.
---
 arm/graphing/graph_panel.py |    9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/arm/graphing/graph_panel.py b/arm/graphing/graph_panel.py
index c45bd69..fc7f7cf 100644
--- a/arm/graphing/graph_panel.py
+++ b/arm/graphing/graph_panel.py
@@ -187,13 +187,6 @@ class GraphStats:
 
     return DEFAULT_CONTENT_HEIGHT
 
-  def is_visible(self):
-    """
-    True if the stat has content to present, false if it should be hidden.
-    """
-
-    return True
-
   def draw(self, panel, width, height):
     """
     Allows for any custom drawing monitor wishes to append.
@@ -296,7 +289,7 @@ class GraphPanel(panel.Panel):
     if hidden).
     """
 
-    if self.current_display and self.stats[self.current_display].is_visible():
+    if self.current_display:
       return self.stats[self.current_display].get_content_height() + self.graph_height
     else:
       return 0





More information about the tor-commits mailing list