change: Make 'segment' a required MultipartPart parameter.

Change in non-public API.

Author: defnull

CHANGELOG.rst

@@ -16,6 +16,9 @@ Release 1.3
 **Not released yet**
 
 * feat: Nicer error messages when reading from a closed ``MultipartPart``.
+* change: ``segment`` is now a required constructor argument for ``MultipartPart``
+  and changed position. The constructor is not part of the public API, so this
+  should not break user code.
 
 Release 1.2
 ===========

multipart.py

@@ -771,10 +771,10 @@ def _iterparse(self):
                 for event in parser.parse(chunk):
                     if isinstance(event, MultipartSegment):
                         part = MultipartPart(
+                            event,
                             buffer_size=self.buffer_size,
                             memfile_limit=self.spool_limit,
                             charset=self.charset,
-                            segment=event,
                         )
                     elif event:
                         assert part
@@ -804,10 +804,10 @@ class MultipartPart:
 
     def __init__(
         self,
+        segment: "MultipartSegment",
         buffer_size=2**16,
         memfile_limit=2**18,
         charset="utf8",
-        segment: "MultipartSegment" = None,
     ):
 
         """ Private constructor, used by :class:`MultipartParser` """