Description: ------------ As I mailed earlier I modified your DisallowCountInLoopsSniff to disallow frequently used count-ish functions in loop conditions. If someone writes for ( $i = 0; $i < strlen($string); $i++ ) { strlen is computed for every loop iteration, leading to longer run times. This sniff issues a warning and suggests assigning the variable beforehand. Test script: --------------- $string = 'hello'; for ( $i = 0; $i < strlen($string); $i++ ) { print $i; } Expected result: ---------------- See a warning from the sniff Actual result: -------------- The use of sizeof,strlen inside a loop condition reduces performance. Assign the return value of the function to a variable and use the variable in the loop condition instead.

Thanks for the sniff. I've replace the existing DisallowCountInLoops sniff with this new one and added count() to the list of disallowed functions.

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.