Message79902
Thanks, Benjamin! Checked in in r68553, backported to 3.0 in r68556.
Here's the second patch, which fixes almost all remaining uses of nb_long
but stops short of actually removing/renaming the nb_long slot.
Notes:
(1) I haven't tested the change to PC/winreg.c
(2) The Modules/_struct.c change does introduce a change in behaviour:
for example, before the patch,
struct.pack('q', decimal.Decimal(1))
raises struct.error. After the patch, the packing succeeds. I *think*
the patched behaviour is probably the right behaviour, since it agrees
with 2.x, but it's not 100% clear to me what the intentions of the struct
module are with respect to integer packing of non-integer types. This is
probably a question for another issue. |
|
Date |
User |
Action |
Args |
2009-01-15 16:48:16 | mark.dickinson | set | recipients:
+ mark.dickinson, benjamin.peterson |
2009-01-15 16:48:16 | mark.dickinson | set | messageid: <1232038096.18.0.856118615051.issue4910@psf.upfronthosting.co.za> |
2009-01-15 16:48:15 | mark.dickinson | link | issue4910 messages |
2009-01-15 16:48:14 | mark.dickinson | create | |
|