Make BPythonFormatter.f_strings an instance (not class) attribute.

marienz · marienz · commit b2630fcdd9b9 · 2010-02-08T22:14:49.000+01:00
Having this as a class attribute means you can only use one
color_scheme per session, which could be considered a bug.
diff --git a/bpython/formatter.py b/bpython/formatter.py
@@ -83,16 +83,14 @@ class BPythonFormatter(Formatter):
     See the Pygments source for more info; it's pretty
     straightforward."""
 
-    f_strings = {}
-
     def __init__(self, color_scheme, **options):
-        if not self.f_strings:
-            for k, v in theme_map.iteritems():
-                self.f_strings[k] = '\x01%s' % (color_scheme[v],)
-                if k is Parenthesis:
-                    # FIXME: Find a way to make this the inverse of the current
-                    # background colour
-                    self.f_strings[k] += 'I'
+        self.f_strings = {}
+        for k, v in theme_map.iteritems():
+            self.f_strings[k] = '\x01%s' % (color_scheme[v],)
+            if k is Parenthesis:
+                # FIXME: Find a way to make this the inverse of the current
+                # background colour
+                self.f_strings[k] += 'I'
         Formatter.__init__(self, **options)
 
     def format(self, tokensource, outfile):
