Use builtin copy for shallow copy

deep-jkl · deep-jkl · commit f765d048852c · 2021-07-24T17:54:45.000+02:00
diff --git a/lib/matplotlib/path.py b/lib/matplotlib/path.py
@@ -9,6 +9,7 @@
 visualisation.
 """
 
+import copy
 from functools import lru_cache
 from weakref import WeakValueDictionary
 
@@ -259,20 +260,12 @@ def readonly(self):
         """
         return self._readonly
 
-    def __copy__(self):
+    def copy(self):
         """
         Return a shallow copy of the `Path`, which will share the
         vertices and codes with the source `Path`.
         """
-        try:
-            codes = self.codes
-        except AttributeError:
-            codes = None
-        return self.__class__(
-            self.vertices, codes,
-            _interpolation_steps=self._interpolation_steps)
-
-    copy = __copy__
+        return copy.copy(self)
 
     def __deepcopy__(self, memo=None):
         """
@@ -288,7 +281,7 @@ def __deepcopy__(self, memo=None):
             _interpolation_steps=self._interpolation_steps)
 
     deepcopy = __deepcopy__
-
+    
     @classmethod
     def make_compound_path_from_polys(cls, XY):
         """
diff --git a/lib/matplotlib/tests/test_path.py b/lib/matplotlib/tests/test_path.py
@@ -334,11 +334,11 @@ def test_path_deepcopy():
     path2 = Path(verts, codes)
     path1_copy = path1.deepcopy()
     path2_copy = path2.deepcopy()
-    assert(path1 is not path1_copy)
-    assert(path1.vertices is not path1_copy.vertices)
-    assert(path2 is not path2_copy)
-    assert(path2.vertices is not path2_copy.vertices)
-    assert(path2.codes is not path2_copy.codes)
+    assert path1 is not path1_copy
+    assert path1.vertices is not path1_copy.vertices
+    assert path2 is not path2_copy
+    assert path2.vertices is not path2_copy.vertices
+    assert path2.codes is not path2_copy.codes
 
 
 def test_path_shallowcopy():
@@ -349,11 +349,11 @@ def test_path_shallowcopy():
     path2 = Path(verts, codes)
     path1_copy = path1.copy()
     path2_copy = path2.copy()
-    assert(path1 is not path1_copy)
-    assert(path1.vertices is path1_copy.vertices)
-    assert(path2 is not path2_copy)
-    assert(path2.vertices is path2_copy.vertices)
-    assert(path2.codes is path2_copy.codes)
+    assert path1 is not path1_copy
+    assert path1.vertices is path1_copy.vertices
+    assert path2 is not path2_copy
+    assert path2.vertices is path2_copy.vertices
+    assert path2.codes is path2_copy.codes
 
 
 @pytest.mark.parametrize('phi', np.concatenate([
