Created bug report after discussion on irc (just to make a note here): https://bugs.php.net/bug.php?id=55583
2011/9/4 Ferenc Kovacs <tyr...@php.net>: > Hi, > > because I think that the tests is at fault here. > I checked some other cases, where we validate INI variables(through > PHP_INI_MH), and I think that the code is fine. > a similar case which generates "double" warning: > tyrael@sanctuary:~$ php -n -d error_reporting=-1 -d > display_startup_errors=1 -d zlib.output_compression=1 -d > output_handler=1 -r '' > PHP Fatal error: PHP Startup: Cannot use both zlib.output_compression > and output_handler together!! in Unknown on line 0 > > Fatal error: PHP Startup: Cannot use both zlib.output_compression and > output_handler together!! in Unknown on line 0 > > this happens consistently with all current branches and even with 5.2, > so I think that it is the intended behavior. > so changing the test to expect this seemed the logical choice. > > On Sun, Sep 4, 2011 at 6:10 PM, Alexey Shein <con...@gmail.com> wrote: >> Hello, why did you fix the test this way? Why not to fix the code and >> remove excessive warning? >> >> 2011/9/4 Ferenc Kovacs <tyr...@php.net>: >>> tyrael Sun, 04 Sep 2011 14:28:52 +0000 >>> >>> Revision: http://svn.php.net/viewvc?view=revision&revision=316109 >>> >>> Log: >>> passing an invalid session.upload_progress.freq always generates an >>> additional startup error >>> >>> Changed paths: >>> U >>> php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt >>> U php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt >>> >>> Modified: >>> php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt >>> =================================================================== >>> --- >>> php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt >>> 2011-09-04 14:21:27 UTC (rev 316108) >>> +++ >>> php/php-src/branches/PHP_5_4/ext/session/tests/rfc1867_invalid_settings.phpt >>> 2011-09-04 14:28:52 UTC (rev 316109) >>> @@ -9,5 +9,7 @@ >>> var_dump(ini_get("session.upload_progress.freq")); >>> ?> >>> --EXPECTF-- >>> +PHP Warning: PHP Startup: session.upload_progress.freq must be greater >>> than or equal to zero in %s >>> + >>> Warning: PHP Startup: session.upload_progress.freq must be greater than or >>> equal to zero in %s >>> string(%d) "1%" >>> >>> Modified: php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt >>> =================================================================== >>> --- php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt >>> 2011-09-04 14:21:27 UTC (rev 316108) >>> +++ php/php-src/trunk/ext/session/tests/rfc1867_invalid_settings.phpt >>> 2011-09-04 14:28:52 UTC (rev 316109) >>> @@ -9,5 +9,7 @@ >>> var_dump(ini_get("session.upload_progress.freq")); >>> ?> >>> --EXPECTF-- >>> +PHP Warning: PHP Startup: session.upload_progress.freq must be greater >>> than or equal to zero in %s >>> + >>> Warning: PHP Startup: session.upload_progress.freq must be greater than or >>> equal to zero in %s >>> string(%d) "1%" >>> >>> >>> -- >>> PHP CVS Mailing List (http://www.php.net/) >>> To unsubscribe, visit: http://www.php.net/unsub.php >>> >> >> >> >> -- >> Regards, >> Shein Alexey >> > -- Regards, Shein Alexey -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
