Message273094
Right, sorry. I'm suggesting dropping that second assertion entirely; essentially, we're moving the responsibility for and knowledge about overflow checking into the PyMem_* functions/macros (which is where it belongs), and I don't see a need to check the equivalent condition in list_ass_subscript.
If you do keep the second assertion, you should probably drop the `(size_t)` cast, so that we're not comparing signed with unsigned. |
|
Date |
User |
Action |
Args |
2016-08-19 08:29:36 | mark.dickinson | set | recipients:
+ mark.dickinson, tim.peters, rhettinger, martin.panter, xiang.zhang |
2016-08-19 08:29:36 | mark.dickinson | set | messageid: <1471595376.53.0.828619159629.issue27662@psf.upfronthosting.co.za> |
2016-08-19 08:29:36 | mark.dickinson | link | issue27662 messages |
2016-08-19 08:29:36 | mark.dickinson | create | |
|