Apply suggestions from code review

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Author: Byron

gitdb/stream.py

@@ -268,7 +268,7 @@ def read(self, size=-1):
             if tail:
                 # move the window, make it as large as size demands. For code-clarity,
                 # we just take the chunk from our map again instead of reusing the unconsumed
-                # tail. The latter one would safe some memory copying, but we could end up
+                # tail. The latter one would save some memory copying, but we could end up
                 # with not getting enough data uncompressed, so we had to sort that out as well.
                 # Now we just assume the worst case, hence the data is uncompressed and the window
                 # needs to be as large as the uncompressed bytes we want to read.
@@ -313,7 +313,10 @@ def read(self, size=-1):
             consumed = len(indata) - unused_datalen
             self._cbr += consumed
             self._br += len(chunk)
-            dcompdat += chunk
+            if chunk:
+                if not isinstance(dcompdat, bytearray):
+                    dcompdat = bytearray(dcompdat)
+                dcompdat.extend(chunk)
 
             # Stop when we have enough or there is no path to more output.
             # `chunk` may legitimately be empty mid-stream when zlib is

gitdb/test/test_stream.py

@@ -179,7 +179,8 @@ def test_decompress_reader_chunked_read_does_not_terminate_early(self):
         data = b"hello world! " * 1000
         zdata = zlib.compress(data)
 
-        # Loop with a small chunk size to force many sub-_s recursions.
+        # Loop with a small chunk size to force many internal read/decompression
+        # iterations before EOF.
         for chunk_size in (1, 4, 16, 64):
             reader = DecompressMemMapReader(
                 zdata, close_on_deletion=False, size=len(data)