Message123021
Thanks. This nice, clean diff is much more reviewable and it looks like what I expected.
The use of Py_LOCAL_INLINE is new to me since we usually use #define instead, but this has a cleaner look to it. I am unclear on whether all the our target compilers support an inline keyword. If you're sure it works everywhere, that's great. If not, consider going back to ugly defines -- those reliably work everywhere.
Also note that this patch puts a lot of faith in branch prediction. If some target processor doesn't support it, or has limited ability to remember predictions, or mispredicts, then the code will be slower.
That being said, I'm happy with the patch. You have a +1 from me. |
|
Date |
User |
Action |
Args |
2010-12-01 23:16:45 | rhettinger | set | recipients:
+ rhettinger, tim.peters, collinwinter, terry.reedy, amaury.forgeotdarc, mark.dickinson, pitrou, eric.smith, stutzbach |
2010-12-01 23:16:45 | rhettinger | set | messageid: <1291245405.23.0.223483860341.issue9915@psf.upfronthosting.co.za> |
2010-12-01 23:16:43 | rhettinger | link | issue9915 messages |
2010-12-01 23:16:43 | rhettinger | create | |
|