hi, 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.
On Mon, Feb 22, 2010 at 2:21 AM, Raphael Geissert <geiss...@php.net> wrote: > Raphael Geissert wrote: >> >> If there's no objection, I would like to remove them entirely and use >> strtol in php_filter_int. >> > > By the way, I don't think that the fact that strtol is locale-aware should > be a reason to make the change. Since scripts are expected to use the value > returned by filter_var() there should be no problem with the fact that > strings such as "100'000" are accepted as a valid integer, under certain > locales. > > Cheers, > -- > Raphael Geissert > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > -- 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