Re: cvs: php-src(PHP_5_3) /ext/standard array.c

From: Date: Sun, 10 May 2009 20:14:57 +0000
Subject: Re: cvs: php-src(PHP_5_3) /ext/standard array.c
References: 1 2  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-57690@lists.php.net to get a copy of this message
2009/5/10 Johannes Schlüter <johannes@php.net>: > On Sun, 2009-05-10 at 16:45 +0000, Etienne Kneuss wrote: >> colder                Sun May 10 16:45:21 2009 UTC >> >>   Modified files:              (Branch: PHP_5_3) >>     /php-src/ext/standard     array.c >>   Log: >>   MFB: Fix #48224 (Remove incorrect shuffle) > > Any chance to cover this in a test? - I know it involves randomness. > > Maybe multiple calls to array_rand() and check if it's always in right > order or something? Yeah, I added a === check which should have a low probability of passing with shuffles > > johannes > > > -- Etienne Kneuss http://www.colder.ch Men never do evil so completely and cheerfully as when they do it from a religious conviction. -- Pascal

« previous php.cvs (#57690) next »