David Bonner dbon...@vmware.com added the comment:
Picking this back up again. There's actually no docs changes necessary...the
docs never mentioned that the module didn't support multiple logical streams,
and I didn't see any other mentions in the docs that seemed to need updating.
I
David Bonner dbon...@vmware.com added the comment:
Thanks for the reply.
My company's legal dept. told me that we needed to put the boilerplate
into the files as part of releasing it under the apache license. I used
a tarball because they also recommended including a full copy of the
license
David Bonner dbon...@vmware.com added the comment:
I can remove the boilerplate from the code as long as I add the
following to the submittal:
VMware, Inc. is providing this bz2 module patch to you under the terms
of the Apache License 2.0 with the understanding that you plan to
re-license
David Bonner dbon...@vmware.com added the comment:
Hrm...yeah, I should probably be setting it to closed as soon as
BZ2_bzReadClose() returns, and then back to open once BZ2_bzReadOpen
succeeds. Wasn't intentional...thanks for the catch. You guys need a
new patch with that change in it?
I'll
David Bonner dbon...@vmware.com added the comment:
Understandable. New patch attached.
--
Added file: http://bugs.python.org/file15177/py3k_bz2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue1625
David Bonner dbon...@vmware.com added the comment:
That was mostly just out of paranoia, since the comments mentioned
multiple calls to close being legal. Looking at it again, that particular
case isn't an issue, since we don't hit that call when the mode is
MODE_CLOSED. The testsuite runs