]> granicus.if.org Git - python/commit
Try to squash struct.pack warnings on the "amd64 gentoo trunk"
authorTim Peters <tim.peters@gmail.com>
Wed, 2 Aug 2006 04:12:36 +0000 (04:12 +0000)
committerTim Peters <tim.peters@gmail.com>
Wed, 2 Aug 2006 04:12:36 +0000 (04:12 +0000)
commit62decc9f4966c060a3d4bcf762d8940d666496c5
treeee10fe1ca87904bfeedfd53f6b649a88b170fa2a
parent4edcba69f34ffa09e7e1cc911ee36cbd3fbc14ec
Try to squash struct.pack warnings on the "amd64 gentoo trunk"
buildbot (& possibly other 64-bit boxes) during test_gzip.

The native zlib crc32 function returns an unsigned 32-bit integer,
which the Python wrapper implicitly casts to C long.  Therefore the
same crc can "look negative" on a 32-bit box but "look positive" on
a 64-bit box.  This patch papers over that platform difference when
writing the crc to file.

It may be better to change the Python wrapper, either to make
the result "look positive" on all platforms (which means it may
have to return a Python long at times on a 32-bit box), or to
keep the sign the same across boxes.  But that would be a visible
change in what users see, while the current hack changes no
visible behavior (well, apart from stopping the struct deprecation
warning).

Note that the module-level write32() function is no longer used.
Lib/gzip.py