On Sun, Aug 14, 2011 at 1:24 PM, Derick Rethans <der...@php.net> wrote: > On Sat, 13 Aug 2011, Laruence wrote: > >> Dear all: >> I am going to close strn(case)cmp supporting negative length vote, >> since it has been calling for vote near two weeks, and no new feedback >> . >> >> the Voting result is: >> Support : 6 felipe pajoye pierrick gwynne tyrael laruence >> Decline : 3 iliaa rasmus salathe >> >> it wins 2/3 vote, so I think this supposed to mean that accept, right? > > I voted against too. Also, you started the vote with not even a week > between RFC announcement and call for voting, so I guess that makes this > invalid? > > > Just looked over the RFC, and this whole example is weird: > > <?php > var_dump(strncmp("prefix_num", "num", -10)); > ?> > > Why does it even find the substring as you can't do "-10" from the end? > If the number is too high, it should give you a warning.
imo the patch is consistent with how substr works: tyrael@thor:~$ php -d display_errors=1 -d error_reporting=-1 -r 'echo substr("prefix_num", -100);' prefix_num -- Ferenc Kovács @Tyr43l - http://tyrael.hu -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php