Message380649
> I do not think that this is suitable for newcomers because you need to have deep understanding why it was written in such form at first place and what will be changed if you change it.
I agree contributors need to understand code, rather than simply replace the type. Maybe two weeks is enough to understand the code.
> And it could negatively affect performance, especially on 32-bit platforms.
`long` type can be replaced by `ssize_t`.
`unsigned long` type can be replaced by `size_t`.
And use `PyLong_FromSize_t`/`PyLong_FromSize_t`, then there is no negative impact.
> I don't think that it's worth it to optimize this one.
Although the speedup is small, it's free.
I don't see it as optimization, just no more waste.
> I suggest to fix in it bpo-38252.
I forgot it in that issue, I just searched "0x80808080" in the code, it was missed. |
|
Date |
User |
Action |
Args |
2020-11-10 10:33:09 | malin | set | recipients:
+ malin, paul.moore, vstinner, tim.golden, zach.ware, serhiy.storchaka, steve.dower |
2020-11-10 10:33:09 | malin | set | messageid: <1605004389.88.0.175850249933.issue42304@roundup.psfhosted.org> |
2020-11-10 10:33:09 | malin | link | issue42304 messages |
2020-11-10 10:33:09 | malin | create | |
|