On Tue, Feb 23, 2010 at 12:13 AM, Raphael Geissert <geiss...@php.net> wrote: > Derick Rethans wrote: >> On Mon, 22 Feb 2010, Pierre Joye wrote: >>> Please don't commit this change. I remember some issues with strtol >>> (portability, BC breaks, etc.). Please wait until we figured them out, >>> or find back in the archives why strtol could be a bad idea. >> >> Poratability was indeed the reason, as well as the locale issues that >> you mention (POSIX locales==useless). Please don't commit this. >> > > Ok. Sean came up with a different patch which is being tested right now. > If all the tests pass then I'm going to commit it.
What patch? Please do not commit anything there without first posting to the list. There were enough breakage in this area, no need to introduce new ones again. Thanks for your understanding, Cheers, -- Pierre @pierrejoye | http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
