Just pass the iterator to the constructor, instead of list.extend

Author: JamesParrott

src/shapefile.py

@@ -3425,9 +3425,7 @@ def shapes(self, bbox: BBox | None = None) -> Shapes:
         To only read shapes within a given spatial region, specify the 'bbox'
         arg as a list or tuple of xmin,ymin,xmax,ymax.
         """
-        shapes = Shapes()
-        shapes.extend(self.iterShapes(bbox=bbox))
-        return shapes
+        return Shapes(self.iterShapes(bbox=bbox))
 
     def iterShapes(self, bbox: BBox | None = None) -> Iterator[Shape | None]:
         """Returns a generator of shapes in a shapefile. Useful