Message175846
Patch looks correct and looks good to me, modulo a couple of nitpicks (see Rietveld comments). This seems like a nice cleanup.
The patch introduces a new dependence on PY_UINT32_T, which is something we haven't so far used elsewhere in Python beyond the float<->string conversions. That's quite a big change: it means that it's conceivable that the random module could now fail to build on platforms where it used to work.
It also changes the signature of 'init_by_array', which is one of the core routines taken directly from the MT implementation. |
|
Date |
User |
Action |
Args |
2012-11-18 08:32:41 | mark.dickinson | set | recipients:
+ mark.dickinson, loewis, rhettinger, serhiy.storchaka |
2012-11-18 08:32:41 | mark.dickinson | set | messageid: <1353227561.58.0.281961207841.issue16496@psf.upfronthosting.co.za> |
2012-11-18 08:32:41 | mark.dickinson | link | issue16496 messages |
2012-11-18 08:32:41 | mark.dickinson | create | |
|