On Thu, 2008-11-27 at 13:50 +0000, "Ant Phillips" wrote:
> ant           Thu Nov 27 13:50:28 2008 UTC
> 
>   Modified files:              (Branch: PHP_5_3)
>     /php-src/ext/session/tests        session_save_path_error.phpt 
>                                       session_save_path_variation4.phpt 
>                                       session_name_error.phpt 
>   Log:
>   Latest and greatest versions of these session tests - checked on 5.3 snap 
> on Windows, Linux and Linux 64 bit.
>   
> http://cvs.php.net/viewvc.cgi/php-src/ext/session/tests/session_save_path_error.phpt?r1=1.1.4.4&r2=1.1.4.5&diff_format=u
> Index: php-src/ext/session/tests/session_save_path_error.phpt
> diff -u php-src/ext/session/tests/session_save_path_error.phpt:1.1.4.4 
> php-src/ext/session/tests/session_save_path_error.phpt:1.1.4.5
> --- php-src/ext/session/tests/session_save_path_error.phpt:1.1.4.4    Sat Jun 
> 21 15:27:34 2008
> +++ php-src/ext/session/tests/session_save_path_error.phpt    Thu Nov 27 
> 13:50:28 2008
> @@ -171,8 +171,6 @@
>  string(0) ""
>  
>  -- Iteration 24 --
> -
> -Warning: session_save_path() expects parameter 1 to be string, resource 
> given in %s on line %d
> -NULL
> +string(0) ""
>  Done

This fails on my Linux box after the change and the error message is
what I expect (session_save_path(fopen(__FILE__, "r")) is the relevant
code)

[...]

> http://cvs.php.net/viewvc.cgi/php-src/ext/session/tests/session_name_error.phpt?r1=1.1.4.5&r2=1.1.4.6&diff_format=u
> Index: php-src/ext/session/tests/session_name_error.phpt
> diff -u php-src/ext/session/tests/session_name_error.phpt:1.1.4.5 
> php-src/ext/session/tests/session_name_error.phpt:1.1.4.6
> --- php-src/ext/session/tests/session_name_error.phpt:1.1.4.5 Sun Jun 22 
> 19:16:44 2008
> +++ php-src/ext/session/tests/session_name_error.phpt Thu Nov 27 13:50:28 2008
> @@ -167,8 +167,6 @@
>  string(0) ""
>  
>  -- Iteration 24 --
> -
> -Warning: session_name() expects parameter 1 to be string, resource given in 
> %s on line %d
> -NULL
> +string(0) ""
>  Done

here the same.

Could you please double check?

Thanks,
johannes


-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to