Commit a05f6e24 authored by Tim Peters's avatar Tim Peters

_Stream.close(): Try to kill struct.pack() warnings when

writing the crc to file on the "PPC64 Debian trunk" buildbot
when running test_tarfile.

This is again a case where the native zlib crc is an unsigned
32-bit int, but the Python wrapper implicitly casts it to
signed C long, so that "the sign bit looks different" on
different platforms.
parent 99dfe3c4
...@@ -417,7 +417,13 @@ class _Stream: ...@@ -417,7 +417,13 @@ class _Stream:
self.fileobj.write(self.buf) self.fileobj.write(self.buf)
self.buf = "" self.buf = ""
if self.comptype == "gz": if self.comptype == "gz":
self.fileobj.write(struct.pack("<l", self.crc)) # The native zlib crc is an unsigned 32-bit integer, but
# the Python wrapper implicitly casts that to a signed C
# long. So, on a 32-bit box self.crc may "look negative",
# while the same crc on a 64-bit box may "look positive".
# To avoid irksome warnings from the `struct` module, force
# it to look positive on all boxes.
self.fileobj.write(struct.pack("<L", self.crc & 0xffffffffL))
self.fileobj.write(struct.pack("<L", self.pos & 0xffffFFFFL)) self.fileobj.write(struct.pack("<L", self.pos & 0xffffFFFFL))
if not self._extfileobj: if not self._extfileobj:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment