Change update methods to only cache information and compute minimal things.

Add a format() method that actually do the heavy duty lifting,
such as string formatting, which is only needed during actual
output.

1 files changed, 11 insertions, 11 deletions

diff --git a/ttystatus/index.py b/ttystatus/index.py
index a9e1541..941165a 100644
--- a/ttystatus/index.py
+++ b/ttystatus/index.py
@@ -24,18 +24,18 @@ class Index(ttystatus.Widget):
     def __init__(self, name, listname):
         self.name = name
         self.listname = listname
-        self.value = self.format(0, 0)
         self.interesting_keys = [name, listname]
+        self.value = None
+        self.listvalue = []
         
-    def format(self, index, listlen):
-        return '%d/%d' % (index, listlen)
-        
-    def update(self, master, width):
-        value = master[self.name]
-        listvalue = master[self.listname]
+    def format(self):
         try:
-            index = listvalue.index(value) + 1
+            index = self.listvalue.index(self.value) + 1
         except ValueError:
-            pass
-        else:
-            self.value = self.format(index, len(listvalue))
+            index = 0
+        return '%d/%d' % (index, len(self.listvalue))
+        
+    def update(self, master, width):
+        self.value = master[self.name]
+        self.listvalue = master[self.listname]
+