Message77324
I think your patch leaks a reference to 'x'. Move the Py_DECREF(x) to
before "if (err < 0)"?
And then really nitpicky: your test has 3 blank lines after it, and
should have 2.
Otherwise looks great. Thanks for picking this up! |
|
Date |
User |
Action |
Args |
2008-12-08 16:40:19 | jyasskin | set | recipients:
+ jyasskin, amaury.forgeotdarc |
2008-12-08 16:40:19 | jyasskin | set | messageid: <1228754419.06.0.475982879058.issue4589@psf.upfronthosting.co.za> |
2008-12-08 16:40:18 | jyasskin | link | issue4589 messages |
2008-12-08 16:40:18 | jyasskin | create | |
|