Small lint-like things

Author: rocky

mathics/builtin/graphics.py

@@ -416,7 +416,6 @@ def apply(self, graphics, evaluation, options):
         new_leaves = []
         options_set = set(options.keys())
         for leaf in graphics.leaves:
-            new_leaf = leaf
             leaf_name = leaf.get_head_name()
             if leaf_name == "System`Rule" and str(leaf.leaves[0]) in options_set:
                 continue
@@ -1366,64 +1365,6 @@ def _arc_params(self):
 
         return x, y, abs(rx), abs(ry), sx, sy, ex, ey, large_arc
 
-    # FIXME: Why do we need this? If so,
-    # figure out how to put in svg.py
-    # --------------------------------
-    # def to_svg(self, offset=None):
-    #     if self.arc is None:
-    #         return super(_ArcBox, self).to_svg(offset)
-
-    #     x, y, rx, ry, sx, sy, ex, ey, large_arc = self._arc_params()
-
-    #     format_fn = lookup_method(self, "svg")
-    #     if format_fn is not None:
-    #         return format_fn(self, offset)
-    #     def path(closed):
-    #         if closed:
-    #             yield "M %f,%f" % (x, y)
-    #             yield "L %f,%f" % (sx, sy)
-    #         else:
-    #             yield "M %f,%f" % (sx, sy)
-
-    #         yield "A %f,%f,0,%d,0,%f,%f" % (rx, ry, large_arc, ex, ey)
-
-    #         if closed:
-    #             yield "Z"
-
-    #     l = self.style.get_line_width(face_element=self.face_element)
-    #     style = create_css(self.edge_color, self.face_color, stroke_width=l)
-    #     return '<path d="%s" style="%s" />' % (" ".join(path(self.face_element)), style)
-
-    def to_asy(self):
-        if self.arc is None:
-            return super(_ArcBox, self).to_asy()
-
-        x, y, rx, ry, sx, sy, ex, ey, large_arc = self._arc_params()
-
-        def path(closed):
-            if closed:
-                yield "(%s,%s)--(%s,%s)--" % tuple(
-                    asy_number(t) for t in (x, y, sx, sy)
-                )
-
-            yield "arc((%s,%s), (%s, %s), (%s, %s))" % tuple(
-                asy_number(t) for t in (x, y, sx, sy, ex, ey)
-            )
-
-            if closed:
-                yield "--cycle"
-
-        l = self.style.get_line_width(face_element=self.face_element)
-        pen = create_pens(
-            edge_color=self.edge_color,
-            face_color=self.face_color,
-            stroke_width=l,
-            is_face_element=self.face_element,
-        )
-        command = "filldraw" if self.face_element else "draw"
-        return "%s(%s, %s);" % (command, "".join(path(self.face_element)), pen)
-
-
 class DiskBox(_ArcBox):
     face_element = True
 

mathics/formatter/svg.py

@@ -76,8 +76,8 @@ def polygon(points):
 
 
 def beziercurvebox(self, offset=None):
-    l = self.style.get_line_width(face_element=False)
-    style = create_css(edge_color=self.edge_color, stroke_width=l)
+    line_width = self.style.get_line_width(face_element=False)
+    style = create_css(edge_color=self.edge_color, stroke_width=line_width)
 
     svg = ""
     for line in self.lines:
@@ -91,9 +91,9 @@ def beziercurvebox(self, offset=None):
 
 
 def filledcurvebox(self, offset=None):
-    l = self.style.get_line_width(face_element=False)
+    line_width = self.style.get_line_width(face_element=False)
     style = create_css(
-        edge_color=self.edge_color, face_color=self.face_color, stroke_width=l
+        edge_color=self.edge_color, face_color=self.face_color, stroke_width=line_width
     )
 
     def components():
@@ -110,7 +110,6 @@ def components():
 
 add_conversion_fn(FilledCurveBox)
 
-# FIXME figure out how we can add this.
 def graphicsbox(self, leaves=None, **options) -> str:
         if not leaves:
             leaves = self._leaves
@@ -220,8 +219,8 @@ def insetbox(self, offset=None):
 
 
 def linebox(self, offset=None):
-    l = self.style.get_line_width(face_element=False)
-    style = create_css(edge_color=self.edge_color, stroke_width=l)
+    line_width = self.style.get_line_width(face_element=False)
+    style = create_css(edge_color=self.edge_color, stroke_width=line_width)
     svg = ""
     for line in self.lines:
         svg += '<polyline points="%s" style="%s" />' % (
@@ -261,13 +260,13 @@ def pointbox(self, offset=None):
 
 
 def polygonbox(self, offset=None):
-    l = self.style.get_line_width(face_element=True)
+    line_width = self.style.get_line_width(face_element=True)
     if self.vertex_colors is None:
         face_color = self.face_color
     else:
         face_color = None
     style = create_css(
-        edge_color=self.edge_color, face_color=face_color, stroke_width=l
+        edge_color=self.edge_color, face_color=face_color, stroke_width=line_width
     )
     svg = ""
     if self.vertex_colors is not None:
@@ -292,7 +291,7 @@ def polygonbox(self, offset=None):
 
 
 def rectanglebox(self, offset=None):
-    l = self.style.get_line_width(face_element=True)
+    line_width = self.style.get_line_width(face_element=True)
     x1, y1 = self.p1.pos()
     x2, y2 = self.p2.pos()
     xmin = min(x1, x2)
@@ -302,7 +301,7 @@ def rectanglebox(self, offset=None):
     if offset:
         x1, x2 = x1 + offset[0], x2 + offset[0]
         y1, y2 = y1 + offset[1], y2 + offset[1]
-    style = create_css(self.edge_color, self.face_color, l)
+    style = create_css(self.edge_color, self.face_color, line_width)
     return '<rect x="%f" y="%f" width="%f" height="%f" style="%s" />' % (
         xmin,
         ymin,
@@ -321,8 +320,8 @@ def _roundbox(self, offset=None):
     rx, ry = self.r.pos()
     rx -= x
     ry = y - ry
-    l = self.style.get_line_width(face_element=self.face_element)
-    style = create_css(self.edge_color, self.face_color, stroke_width=l)
+    line_width = self.style.get_line_width(face_element=self.face_element)
+    style = create_css(self.edge_color, self.face_color, stroke_width=line_width)
     return '<ellipse cx="%f" cy="%f" rx="%f" ry="%f" style="%s" />' % (
         x,
         y,