Commits

Kevin Hunter committed 8fd582f Draft

Mild cleanup

Remove individual ifs in favor of a loop construct. It's mildly
more extensible.

Comments (0)

Files changed (1)

sphinx/ext/graphviz.py

         node = graphviz()
         node['code'] = dotcode
         node['options'] = []
-        if 'align' in self.options:
-            node['align'] = self.options['align']
-        if 'alt' in self.options:
-            node['alt'] = self.options['alt']
-        if 'caption' in self.options:
-            node['caption'] = self.options['caption']
-        node['inline'] = 'inline' in self.options
+        for i in ('align', 'alt', 'caption'):
+            if i in self.options:
+                node[ i ] = self.options[ i ]
+        for i in ('inline',):
+            node[i] = i in self.options
         return [node]
 
 
         node['code'] = '%s %s {\n%s\n}\n' % \
                        (self.name, self.arguments[0], '\n'.join(self.content))
         node['options'] = []
-        if 'align' in self.options:
-            node['align'] = self.options['align']
-        if 'alt' in self.options:
-            node['alt'] = self.options['alt']
-        if 'caption' in self.options:
-            node['caption'] = self.options['caption']
-        node['inline'] = 'inline' in self.options
+        for i in ('align', 'alt', 'caption'):
+            if i in self.options:
+                node[ i ] = self.options[ i ]
+        for i in ('inline',):
+            node[i] = i in self.options
         return [node]