Do not double the pattern length

Author: radarhere

Tests/test_imagedraw.py

@@ -1783,20 +1783,6 @@ def test_line_dash_multi_segment() -> None:
     assert im.getbbox() is not None
 
 
-def test_line_dash_odd_pattern() -> None:
-    # An odd-length dash pattern should be doubled per SVG spec
-    im = Image.new("RGB", (W, H))
-    draw = ImageDraw.Draw(im)
-    draw.line([(10, 50), (90, 50)], "yellow", 2, dash=(10,))
-
-    expected = Image.new("RGB", (W, H))
-    draw2 = ImageDraw.Draw(expected)
-    draw2.line([(10, 50), (90, 50)], "yellow", 2, dash=(10, 10))
-
-    # odd pattern (10,) becomes (10, 10)
-    assert_image_equal(im, expected)
-
-
 def test_line_dash_empty() -> None:
     im = Image.new("RGB", (W, H))
     draw = ImageDraw.Draw(im)

src/PIL/ImageDraw.py

@@ -315,9 +315,6 @@ def line(
             if len(dash) == 0:
                 msg = "dash must be a non-empty tuple of ints"
                 raise ValueError(msg)
-            # If odd number of elements, double the pattern per SVG spec
-            if len(dash) % 2 != 0:
-                dash *= 2
             points = self._normalize_points(xy)
             dash_offset = 0
             for i in range(len(points) - 1):
@@ -440,8 +437,6 @@ def polygon(
             if len(dash) == 0:
                 msg = "dash must be a non-empty tuple of ints"
                 raise ValueError(msg)
-            if len(dash) % 2 != 0:
-                dash *= 2
             points = self._normalize_points(xy)
             # Close the polygon by connecting last point to first
             if points[0] != points[-1]:
@@ -503,10 +498,8 @@ def rectangle(
                 (x0, y1),
                 (x0, y0),
             ]
-            if len(dash) % 2 != 0:
-                dash *= 2
             dash_offset = 0
-            for i in range(len(rect_points) - 1):
+            for i in range(4):
                 dash_offset = self._draw_dashed_line(
                     rect_points[i],
                     rect_points[i + 1],