[PHP-CVS] cvs: php-src /ext/xsl php_xsl.c php_xsl.h xsl_fe.h xsltprocessor.c
chregu Tue Oct 2 06:32:17 2007 UTC Modified files: /php-src/ext/xslphp_xsl.c php_xsl.h xsl_fe.h xsltprocessor.c Log: - Added xsl->setProfiling() for profiling stylesheets. (MFB) http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/php_xsl.c?r1=1.43&r2=1.44&diff_format=u Index: php-src/ext/xsl/php_xsl.c diff -u php-src/ext/xsl/php_xsl.c:1.43 php-src/ext/xsl/php_xsl.c:1.44 --- php-src/ext/xsl/php_xsl.c:1.43 Fri Sep 28 05:32:41 2007 +++ php-src/ext/xsl/php_xsl.c Tue Oct 2 06:32:16 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: php_xsl.c,v 1.43 2007/09/28 05:32:41 tony2001 Exp $ */ +/* $Id: php_xsl.c,v 1.44 2007/10/02 06:32:16 chregu Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -123,6 +123,7 @@ intern->registered_phpfunctions = NULL; intern->node_list = NULL; intern->doc = NULL; + intern->profiling = NULL; zend_object_std_init(&intern->std, class_type TSRMLS_CC); zend_hash_copy(intern->std.properties, &class_type->default_properties, (copy_ctor_func_t) zval_add_ref, (void *) &tmp, sizeof(zval *)); http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/php_xsl.h?r1=1.17&r2=1.18&diff_format=u Index: php-src/ext/xsl/php_xsl.h diff -u php-src/ext/xsl/php_xsl.h:1.17 php-src/ext/xsl/php_xsl.h:1.18 --- php-src/ext/xsl/php_xsl.h:1.17 Mon Jan 1 09:29:34 2007 +++ php-src/ext/xsl/php_xsl.h Tue Oct 2 06:32:16 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: php_xsl.h,v 1.17 2007/01/01 09:29:34 sebastian Exp $ */ +/* $Id: php_xsl.h,v 1.18 2007/10/02 06:32:16 chregu Exp $ */ #ifndef PHP_XSL_H #define PHP_XSL_H @@ -60,6 +60,7 @@ HashTable *registered_phpfunctions; HashTable *node_list; php_libxml_node_object *doc; + char *profiling; } xsl_object; void php_xsl_set_object(zval *wrapper, void *obj TSRMLS_DC); http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/xsl_fe.h?r1=1.12&r2=1.13&diff_format=u Index: php-src/ext/xsl/xsl_fe.h diff -u php-src/ext/xsl/xsl_fe.h:1.12 php-src/ext/xsl/xsl_fe.h:1.13 --- php-src/ext/xsl/xsl_fe.h:1.12 Fri Sep 28 05:32:07 2007 +++ php-src/ext/xsl/xsl_fe.hTue Oct 2 06:32:16 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: xsl_fe.h,v 1.12 2007/09/28 05:32:07 tony2001 Exp $ */ +/* $Id: xsl_fe.h,v 1.13 2007/10/02 06:32:16 chregu Exp $ */ #ifndef XSL_FE_H #define XSL_FE_H @@ -33,4 +33,5 @@ PHP_FUNCTION(xsl_xsltprocessor_remove_parameter); PHP_FUNCTION(xsl_xsltprocessor_has_exslt_support); PHP_FUNCTION(xsl_xsltprocessor_register_php_functions); +PHP_FUNCTION(xsl_xsltprocessor_set_profiling); #endif http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/xsltprocessor.c?r1=1.57&r2=1.58&diff_format=u Index: php-src/ext/xsl/xsltprocessor.c diff -u php-src/ext/xsl/xsltprocessor.c:1.57 php-src/ext/xsl/xsltprocessor.c:1.58 --- php-src/ext/xsl/xsltprocessor.c:1.57Thu Sep 27 18:28:43 2007 +++ php-src/ext/xsl/xsltprocessor.c Tue Oct 2 06:32:16 2007 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: xsltprocessor.c,v 1.57 2007/09/27 18:28:43 dmitry Exp $ */ +/* $Id: xsltprocessor.c,v 1.58 2007/10/02 06:32:16 chregu Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -44,6 +44,7 @@ PHP_FALIAS(removeParameter, xsl_xsltprocessor_remove_parameter, NULL) PHP_FALIAS(hasExsltSupport, xsl_xsltprocessor_has_exslt_support, NULL) PHP_FALIAS(registerPHPFunctions, xsl_xsltprocessor_register_php_functions, NULL) + PHP_FALIAS(setProfiling, xsl_xsltprocessor_set_profiling, NULL) {NULL, NULL, NULL} }; @@ -425,6 +426,7 @@ int clone; zval *doXInclude, *member; zend_object_handlers *std_hnd; + FILE *f; node = php_libxml_import_node(docp TSRMLS_CC); @@ -440,6 +442,17 @@ php_error_docref(NULL TSRMLS_CC, E_WARNING, "No stylesheet associated to this object"); return NULL; } + + if (intern->profiling) { + if (php_check_open_basedir(intern->profiling TSRMLS_CC)) { + f = NULL; + } else { + f = VCWD_FOPEN(intern->profiling, "w"); + } + } else { + f = NULL; + } + if (intern->parameter) { params = php_xsl_xslt_make_params(intern->parameter, 0 TSRMLS_CC); } @@ -470,8 +483,10 @@ } efree(member); - newdocp = xsltApplyStylesheetUser(style, doc, (const char**) params, NULL, NULL, ctxt); - + newdocp = xsltApplyStylesheetUser(style, doc, (const char**) params, NULL, f, ctxt); + if (f) { + fclose(f); + } xsltFreeTransformContext(ctxt); if (intern->node_list != NULL) {
[PHP-CVS] cvs: php-src(PHP_5_3) / NEWS /ext/xsl php_xsl.c php_xsl.h xsl_fe.h xsltprocessor.c
chregu Tue Oct 2 06:31:18 2007 UTC Modified files: (Branch: PHP_5_3) /php-srcNEWS /php-src/ext/xslphp_xsl.c php_xsl.h xsl_fe.h xsltprocessor.c Log: - Added xsl->setProfiling() for profiling stylesheets. http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.965.2.16&r2=1.2027.2.547.2.965.2.17&diff_format=u Index: php-src/NEWS diff -u php-src/NEWS:1.2027.2.547.2.965.2.16 php-src/NEWS:1.2027.2.547.2.965.2.17 --- php-src/NEWS:1.2027.2.547.2.965.2.16Mon Oct 1 12:40:52 2007 +++ php-src/NEWSTue Oct 2 06:31:17 2007 @@ -1,6 +1,7 @@ PHPNEWS ||| ?? ??? 20??, PHP 5.3.0 +- Added xsl->setProfiling() for profiling stylesheets. (Christian) - Added long-option feature to getopt() and made getopt() available also on win32 systems by adding a common getopt implementation to core. (Jani, David Soria Parra) http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/php_xsl.c?r1=1.32.2.6.2.2.2.2&r2=1.32.2.6.2.2.2.3&diff_format=u Index: php-src/ext/xsl/php_xsl.c diff -u php-src/ext/xsl/php_xsl.c:1.32.2.6.2.2.2.2 php-src/ext/xsl/php_xsl.c:1.32.2.6.2.2.2.3 --- php-src/ext/xsl/php_xsl.c:1.32.2.6.2.2.2.2 Fri Sep 28 05:33:11 2007 +++ php-src/ext/xsl/php_xsl.c Tue Oct 2 06:31:17 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: php_xsl.c,v 1.32.2.6.2.2.2.2 2007/09/28 05:33:11 tony2001 Exp $ */ +/* $Id: php_xsl.c,v 1.32.2.6.2.2.2.3 2007/10/02 06:31:17 chregu Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -123,6 +123,7 @@ intern->registered_phpfunctions = NULL; intern->node_list = NULL; intern->doc = NULL; + intern->profiling = NULL; zend_object_std_init(&intern->std, class_type TSRMLS_CC); zend_hash_copy(intern->std.properties, &class_type->default_properties, (copy_ctor_func_t) zval_add_ref, (void *) &tmp, sizeof(zval *)); http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/php_xsl.h?r1=1.15.2.1.2.1&r2=1.15.2.1.2.1.2.1&diff_format=u Index: php-src/ext/xsl/php_xsl.h diff -u php-src/ext/xsl/php_xsl.h:1.15.2.1.2.1 php-src/ext/xsl/php_xsl.h:1.15.2.1.2.1.2.1 --- php-src/ext/xsl/php_xsl.h:1.15.2.1.2.1 Mon Jan 1 09:36:10 2007 +++ php-src/ext/xsl/php_xsl.h Tue Oct 2 06:31:17 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: php_xsl.h,v 1.15.2.1.2.1 2007/01/01 09:36:10 sebastian Exp $ */ +/* $Id: php_xsl.h,v 1.15.2.1.2.1.2.1 2007/10/02 06:31:17 chregu Exp $ */ #ifndef PHP_XSL_H #define PHP_XSL_H @@ -60,6 +60,7 @@ HashTable *registered_phpfunctions; HashTable *node_list; php_libxml_node_object *doc; + char *profiling; } xsl_object; void php_xsl_set_object(zval *wrapper, void *obj TSRMLS_DC); http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/xsl_fe.h?r1=1.8.2.1.2.1.2.1&r2=1.8.2.1.2.1.2.2&diff_format=u Index: php-src/ext/xsl/xsl_fe.h diff -u php-src/ext/xsl/xsl_fe.h:1.8.2.1.2.1.2.1 php-src/ext/xsl/xsl_fe.h:1.8.2.1.2.1.2.2 --- php-src/ext/xsl/xsl_fe.h:1.8.2.1.2.1.2.1Thu Sep 27 18:00:46 2007 +++ php-src/ext/xsl/xsl_fe.hTue Oct 2 06:31:18 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: xsl_fe.h,v 1.8.2.1.2.1.2.1 2007/09/27 18:00:46 dmitry Exp $ */ +/* $Id: xsl_fe.h,v 1.8.2.1.2.1.2.2 2007/10/02 06:31:18 chregu Exp $ */ #ifndef XSL_FE_H #define XSL_FE_H @@ -33,4 +33,5 @@ PHP_FUNCTION(xsl_xsltprocessor_remove_parameter); PHP_FUNCTION(xsl_xsltprocessor_has_exslt_support); PHP_FUNCTION(xsl_xsltprocessor_register_php_functions); +PHP_FUNCTION(xsl_xsltprocessor_set_profiling); #endif http://cvs.php.net/viewvc.cgi/php-src/ext/xsl/xsltprocessor.c?r1=1.39.2.2.2.9.2.1&r2=1.39.2.2.2.9.2.2&diff_format=u Index: php-src/ext/xsl/xsltprocessor.c diff -u php-src/ext/xsl/xsltprocessor.c:1.39.2.2.2.9.2.1 php-src/ext/xsl/xsltprocessor.c:1.39.2.2.2.9.2.2 --- php-src/ext/xsl/xsltprocessor.c:1.39.2.2.2.9.2.1Thu Sep 27 18:00:46 2007 +++ php-src/ext/xsl/xsltprocessor.c Tue Oct 2 06:31:18 2007 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: xsltprocessor.c,v 1.39.2.2.2.9.2.1 2007/09/27 18:00:46 dmitry Exp $ */ +/* $Id: xsltprocessor.c,v 1.39.2.2.2.9.2.2 2007/10/02 06:31:18 chregu Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -44,6 +44,7 @@ PHP_FALIAS(removeParameter, xsl_xsltprocessor_remove_parameter, NULL) PHP_FALIAS(hasExsltSupport, xsl_xsltprocessor_has_exslt_support, NULL) PHP_FALIAS(registerPHPFunctions, xsl_xsltprocessor_register_php_functions, NULL) + PHP_FALIAS(setProfiling, xsl_xsltprocessor_set_profiling, NULL) {NULL, NULL, NULL} }; @@ -424,6 +425,7 @@ int clone; zval *doXInclude, *member;
Re: [PHP-CVS] cvs: php-src(PHP_5_2) /ext/standard/tests/strings chunk_split_variation2.phpt chunk_split_variation5.phpt chunk_split_variation8.phpt
Hello Antony, this is not about fixing BUGS as in PHP behaves wrong. This is about consistency. We decided on a format for all error messages and Jani was enforcing that. Anyway, it does not matter how long something wrong is in the codebase. Longer errors are not better errors. They only tend to get loved and become features. The real issue here is in fact that you might want to write a strict rule for parsing errors, with some errors having dots and some not it is harder to get the real text. marcus Monday, October 1, 2007, 12:22:43 PM, you wrote: > tony2001Mon Oct 1 10:22:43 2007 UTC > Modified files: (Branch: PHP_5_2) > /php-src/ext/standard/tests/strings chunk_split_variation2.phpt > chunk_split_variation5.phpt > chunk_split_variation8.phpt > Log: > these dots exist since PHP3 > even though Jani removed them in 5_3, I can't see how they can cause any > BUGS > > > http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/chunk_split_variation2.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u > Index: php-src/ext/standard/tests/strings/chunk_split_variation2.phpt > diff -u > php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.1 > php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.2 > --- > php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.1 > Sat Sep 29 12:55:34 2007 > +++ php-src/ext/standard/tests/strings/chunk_split_variation2.phpt Mon > Oct 1 10:22:43 2007 > @@ -95,21 +95,21 @@ > string(28) "This is ch*uklen vari*ation*" > -- Iteration 2 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 3 -- > string(26) "This is chuklen variation*" > -- Iteration 4 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 5 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 6 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 7 -- > string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" > @@ -121,39 +121,39 @@ > string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" > -- Iteration 11 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 12 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 13 -- > string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" > -- Iteration 14 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 15 -- > string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" > -- Iteration 16 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 17 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 18 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 19 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 20 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s on > line %d > bool(false) > -- Iteration 21 -- > > @@ -161,11 +161,11 @@ > string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" > -- Iteration 22 -- > > -Warning: chunk_split(): Chunk length should be greater than zero in %s on > line %d > +Warning: chunk_split(): Chunk length should be greater than zero. in %s
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard basic_functions.c
johannesMon Oct 1 20:36:01 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/standard basic_functions.c Log: - pass the adress not the value http://cvs.php.net/viewvc.cgi/php-src/ext/standard/basic_functions.c?r1=1.725.2.31.2.64.2.4&r2=1.725.2.31.2.64.2.5&diff_format=u Index: php-src/ext/standard/basic_functions.c diff -u php-src/ext/standard/basic_functions.c:1.725.2.31.2.64.2.4 php-src/ext/standard/basic_functions.c:1.725.2.31.2.64.2.5 --- php-src/ext/standard/basic_functions.c:1.725.2.31.2.64.2.4 Mon Oct 1 12:40:53 2007 +++ php-src/ext/standard/basic_functions.c Mon Oct 1 20:36:01 2007 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: basic_functions.c,v 1.725.2.31.2.64.2.4 2007/10/01 12:40:53 jani Exp $ */ +/* $Id: basic_functions.c,v 1.725.2.31.2.64.2.5 2007/10/01 20:36:01 johannes Exp $ */ #include "php.h" #include "php_streams.h" @@ -6125,7 +6125,7 @@ RETURN_FALSE; } - if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss", &path, &path_len, &new_path, new_path_len) == FAILURE) { + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "ss", &path, &path_len, &new_path, &new_path_len) == FAILURE) { return; } -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/json/tests bug42785.phpt
iliaa Mon Oct 1 15:33:29 2007 UTC Modified files: /php-src/ext/json/tests bug42785.phpt Log: MFB: Added test for bug #42785 http://cvs.php.net/viewvc.cgi/php-src/ext/json/tests/bug42785.phpt?r1=1.1&r2=1.2&diff_format=u Index: php-src/ext/json/tests/bug42785.phpt diff -u /dev/null php-src/ext/json/tests/bug42785.phpt:1.2 --- /dev/null Mon Oct 1 15:33:29 2007 +++ php-src/ext/json/tests/bug42785.phptMon Oct 1 15:33:29 2007 @@ -0,0 +1,26 @@ +--TEST-- +Bug #42785 (Incorrect formatting of double values with non-english locales) +--SKIPIF-- + +--FILE-- +a = 100.10; +$bar1->b = "foo"; +var_dump(json_encode($bar1)); +?> +--EXPECT-- +string(13) "[100.1,"bar"]" +string(21) "{"a":100.1,"b":"foo"}" -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/json/tests bug42785.phpt
iliaa Mon Oct 1 15:33:21 2007 UTC Added files: (Branch: PHP_5_2) /php-src/ext/json/tests bug42785.phpt Log: MFB: Added test for bug #42785 http://cvs.php.net/viewvc.cgi/php-src/ext/json/tests/bug42785.phpt?view=markup&rev=1.1 Index: php-src/ext/json/tests/bug42785.phpt +++ php-src/ext/json/tests/bug42785.phpt -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/json/tests bug42785.phpt
iliaa Mon Oct 1 15:33:11 2007 UTC Added files: (Branch: PHP_5_3) /php-src/ext/json/tests bug42785.phpt Log: Added test for bug #42785 http://cvs.php.net/viewvc.cgi/php-src/ext/json/tests/bug42785.phpt?view=markup&rev=1.1 Index: php-src/ext/json/tests/bug42785.phpt +++ php-src/ext/json/tests/bug42785.phpt -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/json json.c /main snprintf.c spprintf.c
iliaa Mon Oct 1 15:25:01 2007 UTC Modified files: /php-src/main snprintf.c spprintf.c /php-src/ext/json json.c Log: MFB: Fixed bug #42785 (json_encode() formats doubles according to locale rather then following standard syntax). http://cvs.php.net/viewvc.cgi/php-src/main/snprintf.c?r1=1.59&r2=1.60&diff_format=u Index: php-src/main/snprintf.c diff -u php-src/main/snprintf.c:1.59 php-src/main/snprintf.c:1.60 --- php-src/main/snprintf.c:1.59Tue Sep 4 11:19:33 2007 +++ php-src/main/snprintf.c Mon Oct 1 15:25:01 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: snprintf.c,v 1.59 2007/09/04 11:19:33 derick Exp $ */ +/* $Id: snprintf.c,v 1.60 2007/10/01 15:25:01 iliaa Exp $ */ #include "php.h" @@ -1051,6 +1051,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -1091,7 +1092,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt == 'H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') prefix_char = *s++; else if (print_sign) http://cvs.php.net/viewvc.cgi/php-src/main/spprintf.c?r1=1.49&r2=1.50&diff_format=u Index: php-src/main/spprintf.c diff -u php-src/main/spprintf.c:1.49 php-src/main/spprintf.c:1.50 --- php-src/main/spprintf.c:1.49Fri Aug 3 14:30:59 2007 +++ php-src/main/spprintf.c Mon Oct 1 15:25:01 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spprintf.c,v 1.49 2007/08/03 14:30:59 tony2001 Exp $ */ +/* $Id: spprintf.c,v 1.50 2007/10/01 15:25:01 iliaa Exp $ */ /* This is the spprintf implementation. * It has emerged from apache snprintf. See original header: @@ -690,6 +690,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -730,7 +731,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt=='H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') prefix_char = *s++; else if (print_sign) http://cvs.php.net/viewvc.cgi/php-src/ext/json/json.c?r1=1.30&r2=1.31&diff_format=u Index: php-src/ext/json/json.c diff -u php-src/ext/json/json.c:1.30 php-src/ext/json/json.c:1.31 --- php-src/ext/json/json.c:1.30Thu Sep 27 18:28:39 2007 +++ php-src/ext/json/json.c Mon Oct 1 15:25:01 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: json.c,v 1.30 2007/09/27 18:28:39 dmitry Exp $ */ +/* $Id: json.c,v 1.31 2007/10/01 15:25:01 iliaa Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -363,9 +363,9 @@ double dbl = Z_DVAL_P(val); if (!zend_isinf(dbl) && !zend_isnan(dbl)) { - len = spprintf(&d, 0, "%.*g", (int) EG(precision), dbl); - smart_str_appendl(buf, d, len); - efree(d); + len = spprintf(&d, 0, "%.*k", (int) EG(precision), dbl); + smart_str_appendl(buf, d, len); + efree(d); } else { zend_error(E_WARNING, "[json] (json_encode_r) double %.9g does not conform to the JSON spec, encoded as 0.", dbl); smart_str_appendc(buf, '0'); -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) / NEWS /ext/json json.c /main snprintf.c spprintf.c
iliaa Mon Oct 1 15:23:15 2007 UTC Modified files: (Branch: PHP_5_2) /php-src/main snprintf.c spprintf.c /php-src/ext/json json.c /php-srcNEWS Log: MFB: Fixed bug #42785 (json_encode() formats doubles according to locale rather then following standard syntax). http://cvs.php.net/viewvc.cgi/php-src/main/snprintf.c?r1=1.37.2.4.2.14&r2=1.37.2.4.2.15&diff_format=u Index: php-src/main/snprintf.c diff -u php-src/main/snprintf.c:1.37.2.4.2.14 php-src/main/snprintf.c:1.37.2.4.2.15 --- php-src/main/snprintf.c:1.37.2.4.2.14 Fri Aug 3 14:31:27 2007 +++ php-src/main/snprintf.c Mon Oct 1 15:23:15 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: snprintf.c,v 1.37.2.4.2.14 2007/08/03 14:31:27 tony2001 Exp $ */ +/* $Id: snprintf.c,v 1.37.2.4.2.15 2007/10/01 15:23:15 iliaa Exp $ */ #include "php.h" @@ -1004,6 +1004,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -1045,7 +1046,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt=='H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') { prefix_char = *s++; } else if (print_sign) { http://cvs.php.net/viewvc.cgi/php-src/main/spprintf.c?r1=1.25.2.2.2.10&r2=1.25.2.2.2.11&diff_format=u Index: php-src/main/spprintf.c diff -u php-src/main/spprintf.c:1.25.2.2.2.10 php-src/main/spprintf.c:1.25.2.2.2.11 --- php-src/main/spprintf.c:1.25.2.2.2.10 Fri Aug 3 14:31:28 2007 +++ php-src/main/spprintf.c Mon Oct 1 15:23:15 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spprintf.c,v 1.25.2.2.2.10 2007/08/03 14:31:28 tony2001 Exp $ */ +/* $Id: spprintf.c,v 1.25.2.2.2.11 2007/10/01 15:23:15 iliaa Exp $ */ /* This is the spprintf implementation. * It has emerged from apache snprintf. See original header: @@ -600,6 +600,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -640,7 +641,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt=='H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') prefix_char = *s++; else if (print_sign) http://cvs.php.net/viewvc.cgi/php-src/ext/json/json.c?r1=1.9.2.19&r2=1.9.2.20&diff_format=u Index: php-src/ext/json/json.c diff -u php-src/ext/json/json.c:1.9.2.19 php-src/ext/json/json.c:1.9.2.20 --- php-src/ext/json/json.c:1.9.2.19Tue Jul 24 22:57:13 2007 +++ php-src/ext/json/json.c Mon Oct 1 15:23:15 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: json.c,v 1.9.2.19 2007/07/24 22:57:13 bjori Exp $ */ +/* $Id: json.c,v 1.9.2.20 2007/10/01 15:23:15 iliaa Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -359,7 +359,7 @@ double dbl = Z_DVAL_P(val); if (!zend_isinf(dbl) && !zend_isnan(dbl)) { - len = spprintf(&d, 0, "%.*g", (int) EG(precision), dbl); + len = spprintf(&d, 0, "%.*k", (int) EG(precision), dbl); smart_str_appendl(buf, d, len); efree(d); } else { http://cvs.php.net/viewvc.cgi/php-src/NEWS?r1=1.2027.2.547.2.965&r2=1.2027.2.547.2.966&diff_format=u Index: php-src/NEWS diff -u php-src/NEWS:1.2027.2.547.2.965 php-src/NEWS:1.2027.2.547.2.966 --- php-src/NEWS:1.2027.2.547.2.965 Wed Sep 26 15:44:15 2007 +++ php-src/NEWSMon Oct 1 15:23:15 2007 @@ -30,6 +30,8 @@ Reported by Laurent gaffie. (Ilia) - Fixed imagerectangle regression with 1x1 rectangle (libgd #106). (Pierre) +- Fixed bug #42785 (json_encode() formats doubles according to local
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/json json.c /main snprintf.c spprintf.c
iliaa Mon Oct 1 15:22:41 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/main snprintf.c spprintf.c /php-src/ext/json json.c Log: Fixed bug #42785 (json_encode() formats doubles according to locale rather then following standard syntax). http://cvs.php.net/viewvc.cgi/php-src/main/snprintf.c?r1=1.37.2.4.2.14&r2=1.37.2.4.2.14.2.1&diff_format=u Index: php-src/main/snprintf.c diff -u php-src/main/snprintf.c:1.37.2.4.2.14 php-src/main/snprintf.c:1.37.2.4.2.14.2.1 --- php-src/main/snprintf.c:1.37.2.4.2.14 Fri Aug 3 14:31:27 2007 +++ php-src/main/snprintf.c Mon Oct 1 15:22:41 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: snprintf.c,v 1.37.2.4.2.14 2007/08/03 14:31:27 tony2001 Exp $ */ +/* $Id: snprintf.c,v 1.37.2.4.2.14.2.1 2007/10/01 15:22:41 iliaa Exp $ */ #include "php.h" @@ -1004,6 +1004,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -1045,7 +1046,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt=='H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') { prefix_char = *s++; } else if (print_sign) { http://cvs.php.net/viewvc.cgi/php-src/main/spprintf.c?r1=1.25.2.2.2.10&r2=1.25.2.2.2.10.2.1&diff_format=u Index: php-src/main/spprintf.c diff -u php-src/main/spprintf.c:1.25.2.2.2.10 php-src/main/spprintf.c:1.25.2.2.2.10.2.1 --- php-src/main/spprintf.c:1.25.2.2.2.10 Fri Aug 3 14:31:28 2007 +++ php-src/main/spprintf.c Mon Oct 1 15:22:41 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: spprintf.c,v 1.25.2.2.2.10 2007/08/03 14:31:28 tony2001 Exp $ */ +/* $Id: spprintf.c,v 1.25.2.2.2.10.2.1 2007/10/01 15:22:41 iliaa Exp $ */ /* This is the spprintf implementation. * It has emerged from apache snprintf. See original header: @@ -600,6 +600,7 @@ case 'g': + case 'k': case 'G': case 'H': switch(modifier) { @@ -640,7 +641,7 @@ lconv = localeconv(); } #endif - s = php_gcvt(fp_num, precision, *fmt=='H' ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); + s = php_gcvt(fp_num, precision, (*fmt=='H' || *fmt == 'k') ? '.' : LCONV_DECIMAL_POINT, (*fmt == 'G' || *fmt == 'H')?'E':'e', &num_buf[1]); if (*s == '-') prefix_char = *s++; else if (print_sign) http://cvs.php.net/viewvc.cgi/php-src/ext/json/json.c?r1=1.9.2.19.2.1&r2=1.9.2.19.2.2&diff_format=u Index: php-src/ext/json/json.c diff -u php-src/ext/json/json.c:1.9.2.19.2.1 php-src/ext/json/json.c:1.9.2.19.2.2 --- php-src/ext/json/json.c:1.9.2.19.2.1Thu Sep 27 18:00:39 2007 +++ php-src/ext/json/json.c Mon Oct 1 15:22:41 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: json.c,v 1.9.2.19.2.1 2007/09/27 18:00:39 dmitry Exp $ */ +/* $Id: json.c,v 1.9.2.19.2.2 2007/10/01 15:22:41 iliaa Exp $ */ #ifdef HAVE_CONFIG_H #include "config.h" @@ -359,7 +359,7 @@ double dbl = Z_DVAL_P(val); if (!zend_isinf(dbl) && !zend_isnan(dbl)) { - len = spprintf(&d, 0, "%.*g", (int) EG(precision), dbl); + len = spprintf(&d, 0, "%.*k", (int) EG(precision), dbl); smart_str_appendl(buf, d, len); efree(d); } else { -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard/tests/general_functions error_get_last.phpt
johannesMon Oct 1 15:01:47 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/standard/tests/general_functions error_get_last.phpt Log: - Fix test http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/general_functions/error_get_last.phpt?r1=1.1.2.2.2.1&r2=1.1.2.2.2.2&diff_format=u Index: php-src/ext/standard/tests/general_functions/error_get_last.phpt diff -u php-src/ext/standard/tests/general_functions/error_get_last.phpt:1.1.2.2.2.1 php-src/ext/standard/tests/general_functions/error_get_last.phpt:1.1.2.2.2.2 --- php-src/ext/standard/tests/general_functions/error_get_last.phpt:1.1.2.2.2.1 Mon Oct 1 12:40:54 2007 +++ php-src/ext/standard/tests/general_functions/error_get_last.phptMon Oct 1 15:01:47 2007 @@ -24,7 +24,7 @@ ["message"]=> string(54) "error_get_last() expects exactly 0 parameters, 1 given" ["file"]=> - string(78) "%s" + string(%i) "%s" ["line"]=> int(4) } @@ -36,7 +36,7 @@ ["message"]=> string(21) "Undefined variable: b" ["file"]=> - string(78) "%s" + string(%i) "%s" ["line"]=> int(7) } -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /main main.c ZendEngine2 zend_ini.c
iliaa Mon Oct 1 14:53:01 2007 UTC Modified files: (Branch: PHP_5_2) /ZendEngine2zend_ini.c /php-src/main main.c Log: MFB: Refactor code to make fixes for coverity issues #385 and #386 a bit more obvious http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_ini.c?r1=1.39.2.2.2.23&r2=1.39.2.2.2.24&diff_format=u Index: ZendEngine2/zend_ini.c diff -u ZendEngine2/zend_ini.c:1.39.2.2.2.23 ZendEngine2/zend_ini.c:1.39.2.2.2.24 --- ZendEngine2/zend_ini.c:1.39.2.2.2.23Thu Sep 27 17:27:15 2007 +++ ZendEngine2/zend_ini.c Mon Oct 1 14:53:01 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: zend_ini.c,v 1.39.2.2.2.23 2007/09/27 17:27:15 iliaa Exp $ */ +/* $Id: zend_ini.c,v 1.39.2.2.2.24 2007/10/01 14:53:01 iliaa Exp $ */ #include "zend.h" #include "zend_qsort.h" @@ -434,14 +434,16 @@ tmp_value_len = 0; } - if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) { - value = 1; - } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) { - value = 1; - } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) { - value = 1; - } else if (tmp_value) { - value = atoi(tmp_value); + if (tmp_value) { + if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) { + value = 1; + } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) { + value = 1; + } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) { + value = 1; + } else (tmp_value) { + value = atoi(tmp_value); + } } else { value = 0; } http://cvs.php.net/viewvc.cgi/php-src/main/main.c?r1=1.640.2.23.2.57&r2=1.640.2.23.2.58&diff_format=u Index: php-src/main/main.c diff -u php-src/main/main.c:1.640.2.23.2.57 php-src/main/main.c:1.640.2.23.2.58 --- php-src/main/main.c:1.640.2.23.2.57 Mon Sep 24 11:53:40 2007 +++ php-src/main/main.c Mon Oct 1 14:53:01 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: main.c,v 1.640.2.23.2.57 2007/09/24 11:53:40 dmitry Exp $ */ +/* $Id: main.c,v 1.640.2.23.2.58 2007/10/01 14:53:01 iliaa Exp $ */ /* {{{ includes */ @@ -218,7 +218,11 @@ static int php_get_display_errors_mode(char *value, int value_length) { int mode; - + + if (!value) { + return PHP_DISPLAY_ERRORS_STDOUT; + } + if (value_length == 2 && !strcasecmp("on", value)) { mode = PHP_DISPLAY_ERRORS_STDOUT; } else if (value_length == 3 && !strcasecmp("yes", value)) { @@ -229,14 +233,13 @@ mode = PHP_DISPLAY_ERRORS_STDERR; } else if (value_length == 6 && !strcasecmp(value, "stdout")) { mode = PHP_DISPLAY_ERRORS_STDOUT; - } else if (value) { + } else { mode = atoi(value); if (mode && mode != PHP_DISPLAY_ERRORS_STDOUT && mode != PHP_DISPLAY_ERRORS_STDERR) { mode = PHP_DISPLAY_ERRORS_STDOUT; } - } else { - mode = PHP_DISPLAY_ERRORS_STDOUT; } + return mode; } /* }}} */ -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /main main.c ZendEngine2 zend_ini.c
iliaa Mon Oct 1 14:51:11 2007 UTC Modified files: (Branch: PHP_5_3) /ZendEngine2zend_ini.c /php-src/main main.c Log: Refactor code to make fixes for coverity issues #385 and #386 a bit more obvious http://cvs.php.net/viewvc.cgi/ZendEngine2/zend_ini.c?r1=1.39.2.2.2.18.2.9&r2=1.39.2.2.2.18.2.10&diff_format=u Index: ZendEngine2/zend_ini.c diff -u ZendEngine2/zend_ini.c:1.39.2.2.2.18.2.9 ZendEngine2/zend_ini.c:1.39.2.2.2.18.2.10 --- ZendEngine2/zend_ini.c:1.39.2.2.2.18.2.9Thu Sep 27 18:00:37 2007 +++ ZendEngine2/zend_ini.c Mon Oct 1 14:51:11 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: zend_ini.c,v 1.39.2.2.2.18.2.9 2007/09/27 18:00:37 dmitry Exp $ */ +/* $Id: zend_ini.c,v 1.39.2.2.2.18.2.10 2007/10/01 14:51:11 iliaa Exp $ */ #include "zend.h" #include "zend_qsort.h" @@ -437,14 +437,16 @@ tmp_value_len = 0; } - if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) { - value = 1; - } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) { - value = 1; - } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) { - value = 1; - } else if (tmp_value) { - value = atoi(tmp_value); + if (tmp_value) { + if (tmp_value_len == 4 && strcasecmp(tmp_value, "true") == 0) { + value = 1; + } else if (tmp_value_len == 3 && strcasecmp(tmp_value, "yes") == 0) { + value = 1; + } else if (tmp_value_len == 2 && strcasecmp(tmp_value, "on") == 0) { + value = 1; + } else (tmp_value) { + value = atoi(tmp_value); + } } else { value = 0; } http://cvs.php.net/viewvc.cgi/php-src/main/main.c?r1=1.640.2.23.2.57.2.1&r2=1.640.2.23.2.57.2.2&diff_format=u Index: php-src/main/main.c diff -u php-src/main/main.c:1.640.2.23.2.57.2.1 php-src/main/main.c:1.640.2.23.2.57.2.2 --- php-src/main/main.c:1.640.2.23.2.57.2.1 Fri Sep 28 02:05:09 2007 +++ php-src/main/main.c Mon Oct 1 14:51:11 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: main.c,v 1.640.2.23.2.57.2.1 2007/09/28 02:05:09 jani Exp $ */ +/* $Id: main.c,v 1.640.2.23.2.57.2.2 2007/10/01 14:51:11 iliaa Exp $ */ /* {{{ includes */ @@ -218,7 +218,11 @@ static int php_get_display_errors_mode(char *value, int value_length) { int mode; - + + if (!value) { + return PHP_DISPLAY_ERRORS_STDOUT; + } + if (value_length == 2 && !strcasecmp("on", value)) { mode = PHP_DISPLAY_ERRORS_STDOUT; } else if (value_length == 3 && !strcasecmp("yes", value)) { @@ -229,14 +233,13 @@ mode = PHP_DISPLAY_ERRORS_STDERR; } else if (value_length == 6 && !strcasecmp(value, "stdout")) { mode = PHP_DISPLAY_ERRORS_STDOUT; - } else if (value) { + } else { mode = atoi(value); if (mode && mode != PHP_DISPLAY_ERRORS_STDOUT && mode != PHP_DISPLAY_ERRORS_STDERR) { mode = PHP_DISPLAY_ERRORS_STDOUT; } - } else { - mode = PHP_DISPLAY_ERRORS_STDOUT; } + return mode; } /* }}} */ -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/standard/tests/strings bug40754.phpt
janiMon Oct 1 13:55:58 2007 UTC Modified files: /php-src/ext/standard/tests/strings bug40754.phpt Log: MFB: fix test http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/bug40754.phpt?r1=1.3&r2=1.4&diff_format=u Index: php-src/ext/standard/tests/strings/bug40754.phpt diff -u php-src/ext/standard/tests/strings/bug40754.phpt:1.3 php-src/ext/standard/tests/strings/bug40754.phpt:1.4 --- php-src/ext/standard/tests/strings/bug40754.phpt:1.3Mon Jun 18 13:39:02 2007 +++ php-src/ext/standard/tests/strings/bug40754.phptMon Oct 1 13:55:58 2007 @@ -28,7 +28,7 @@ ?> --EXPECTF-- string(4) "bcde" -bool(false) +string(6) "abcdex" bool(false) bool(false) @@ -63,7 +63,7 @@ bool(false) --UEXPECTF-- unicode(4) "bcde" -bool(false) +unicode(6) "abcdex" bool(false) bool(false) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src /ext/standard string.c
janiMon Oct 1 13:41:57 2007 UTC Modified files: /php-src/ext/standard string.c Log: MFB: fix bug #42142 and cs+ws http://cvs.php.net/viewvc.cgi/php-src/ext/standard/string.c?r1=1.652&r2=1.653&diff_format=u Index: php-src/ext/standard/string.c diff -u php-src/ext/standard/string.c:1.652 php-src/ext/standard/string.c:1.653 --- php-src/ext/standard/string.c:1.652 Mon Oct 1 12:04:09 2007 +++ php-src/ext/standard/string.c Mon Oct 1 13:41:57 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: string.c,v 1.652 2007/10/01 12:04:09 jani Exp $ */ +/* $Id: string.c,v 1.653 2007/10/01 13:41:57 jani Exp $ */ /* Synced with php 3.0 revision 1.193 1999-06-16 [ssb] */ @@ -3317,7 +3317,7 @@ if (*f > str_codepts || (*f < 0 && -(*f) > str_codepts)) { ret = 0; } else if (*l > str_codepts || (*l < 0 && -(*l) > str_codepts)) { - ret = 0; + *l = str_codepts; } if (((unsigned)(*f) + (unsigned)(*l)) > str_codepts) { *l = str_codepts - *f; @@ -3414,6 +3414,7 @@ f = Z_LVAL_PP(from); } if (argc > 3) { + SEPARATE_ZVAL(len); if (Z_TYPE_PP(len) != IS_ARRAY) { convert_to_long_ex(len); l = Z_LVAL_PP(len); @@ -3425,8 +3426,10 @@ } if (Z_TYPE_PP(str) != IS_ARRAY) { - if ( (argc == 3 && Z_TYPE_PP(from) == IS_ARRAY) || -(argc == 4 && Z_TYPE_PP(from) != Z_TYPE_PP(len)) ) { + if ( + (argc == 3 && Z_TYPE_PP(from) == IS_ARRAY) || + (argc == 4 && Z_TYPE_PP(from) != Z_TYPE_PP(len)) + ) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "'from' and 'len' should be of same type - numerical or array"); RETURN_ZVAL(*str, 1, 0); } @@ -3438,7 +3441,6 @@ } } - if (Z_TYPE_PP(str) != IS_ARRAY) { if (Z_TYPE_PP(from) != IS_ARRAY ) { if (Z_TYPE_PP(repl) == IS_ARRAY) { @@ -7919,12 +7921,12 @@ offset = (offset < 0) ? 0 : offset; } - if(offset > s1_len) { + if (offset > s1_len) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "The start position cannot exceed initial string length"); RETURN_FALSE; } - if(len > s1_len - offset) { + if (len > s1_len - offset) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "The specified segment exceeds string length"); RETURN_FALSE; } -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard string.c
janiMon Oct 1 13:38:15 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/standard string.c Log: - ws + cs + sync HEAD http://cvs.php.net/viewvc.cgi/php-src/ext/standard/string.c?r1=1.445.2.14.2.69.2.2&r2=1.445.2.14.2.69.2.3&diff_format=u Index: php-src/ext/standard/string.c diff -u php-src/ext/standard/string.c:1.445.2.14.2.69.2.2 php-src/ext/standard/string.c:1.445.2.14.2.69.2.3 --- php-src/ext/standard/string.c:1.445.2.14.2.69.2.2 Mon Oct 1 12:05:41 2007 +++ php-src/ext/standard/string.c Mon Oct 1 13:38:14 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: string.c,v 1.445.2.14.2.69.2.2 2007/10/01 12:05:41 jani Exp $ */ +/* $Id: string.c,v 1.445.2.14.2.69.2.3 2007/10/01 13:38:14 jani Exp $ */ /* Synced with php 3.0 revision 1.193 1999-06-16 [ssb] */ @@ -2181,8 +2181,7 @@ if (Z_TYPE_PP(str) == IS_STRING) { if ( - (argc == 3 && Z_TYPE_PP(from) == IS_ARRAY) - || + (argc == 3 && Z_TYPE_PP(from) == IS_ARRAY) || (argc == 4 && Z_TYPE_PP(from) != Z_TYPE_PP(len)) ) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "'from' and 'len' should be of same type - numerical or array "); @@ -2195,7 +2194,6 @@ } } } - if (Z_TYPE_PP(str) != IS_ARRAY) { if (Z_TYPE_PP(from) != IS_ARRAY) { @@ -5127,12 +5125,12 @@ offset = (offset < 0) ? 0 : offset; } - if(offset > s1_len) { + if (offset > s1_len) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "The start position cannot exceed initial string length"); RETURN_FALSE; } - if(len > s1_len - offset) { + if (len > s1_len - offset) { php_error_docref(NULL TSRMLS_CC, E_WARNING, "The length cannot exceed initial string length"); RETURN_FALSE; } -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard array.c php_var.h
janiMon Oct 1 12:34:19 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/standard array.c php_var.h Log: MFH: sync (adds php_prefix_varname() API function) http://cvs.php.net/viewvc.cgi/php-src/ext/standard/array.c?r1=1.308.2.21.2.37&r2=1.308.2.21.2.37.2.1&diff_format=u Index: php-src/ext/standard/array.c diff -u php-src/ext/standard/array.c:1.308.2.21.2.37 php-src/ext/standard/array.c:1.308.2.21.2.37.2.1 --- php-src/ext/standard/array.c:1.308.2.21.2.37Sat Sep 22 15:38:00 2007 +++ php-src/ext/standard/array.cMon Oct 1 12:34:19 2007 @@ -21,7 +21,7 @@ +--+ */ -/* $Id: array.c,v 1.308.2.21.2.37 2007/09/22 15:38:00 iliaa Exp $ */ +/* $Id: array.c,v 1.308.2.21.2.37.2.1 2007/10/01 12:34:19 jani Exp $ */ #include "php.h" #include "php_ini.h" @@ -618,7 +618,7 @@ /* check if comparison function is valid */ #define PHP_ARRAY_CMP_FUNC_CHECK(func_name)\ if (!zend_is_callable(*func_name, 0, NULL)) { \ - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid comparison function.");\ + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid comparison function"); \ BG(user_compare_fci_cache) = old_user_compare_fci_cache; \ BG(user_compare_func_name) = old_compare_func; \ RETURN_FALSE; \ @@ -1317,82 +1317,86 @@ } /* }}} */ +PHPAPI int php_prefix_varname(zval *result, zval *prefix, char *var_name, int var_name_len, zend_bool add_underscore TSRMLS_DC) /* {{{ */ +{ + Z_STRLEN_P(result) = Z_STRLEN_P(prefix) + (add_underscore ? 1 : 0) + var_name_len; + Z_TYPE_P(result) = IS_STRING; + Z_STRVAL_P(result) = emalloc(Z_STRLEN_P(result) + 1); + memcpy(Z_STRVAL_P(result), Z_STRVAL_P(prefix), Z_STRLEN_P(prefix)); + + if (add_underscore) { + Z_STRVAL_P(result)[Z_STRLEN_P(prefix)] = '_'; + } + + memcpy(Z_STRVAL_P(result) + Z_STRLEN_P(prefix) + (add_underscore ? 1 : 0), var_name, var_name_len + 1); + + return SUCCESS; +} +/* }}} */ + /* {{{ proto int extract(array var_array [, int extract_type [, string prefix]]) Imports variables into symbol table from an array */ PHP_FUNCTION(extract) { - zval **var_array, **z_extract_type, **prefix; + zval *var_array, *prefix = NULL; + long extract_type = EXTR_OVERWRITE; zval **entry, *data; char *var_name; - smart_str final_name = {0}; ulong num_key; uint var_name_len; - int var_exists, extract_type, key_type, count = 0; + int var_exists, key_type, count = 0; int extract_refs = 0; HashPosition pos; - switch (ZEND_NUM_ARGS()) { - case 1: - if (zend_get_parameters_ex(1, &var_array) == FAILURE) { - WRONG_PARAM_COUNT; - } - extract_type = EXTR_OVERWRITE; - break; + if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "a|lz/", &var_array, &extract_type, &prefix) == FAILURE) { + return; + } - case 2: - if (zend_get_parameters_ex(2, &var_array, &z_extract_type) == FAILURE) { - WRONG_PARAM_COUNT; - } - convert_to_long_ex(z_extract_type); - extract_type = Z_LVAL_PP(z_extract_type); - extract_refs = (extract_type & EXTR_REFS); - extract_type &= 0xff; - if (extract_type > EXTR_SKIP && extract_type <= EXTR_PREFIX_IF_EXISTS) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Prefix expected to be specified"); - return; - } - break; - - case 3: - if (zend_get_parameters_ex(3, &var_array, &z_extract_type, &prefix) == FAILURE) { - WRONG_PARAM_COUNT; - } - convert_to_long_ex(z_extract_type); - extract_type = Z_LVAL_PP(z_extract_type); - extract_refs = (extract_type & EXTR_REFS); - extract_type &= 0xff; - convert_to_string_ex(prefix); - break; + extract_refs = (extract_type & EXTR_REFS); + extract_type &= 0xff; - default: - WRONG_PARAM_COUNT; - break; - } - if (extract_type < EXTR_OVERWRITE || extract_type > EXTR_IF_EXISTS) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Unknown extract type"); + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid extract type"); return; } -
[PHP-CVS] cvs: php-src / configure.in /ext/standard basic_functions.c config.m4 /main getopt.c php_getopt.h /sapi/cgi cgi_main.c config.w32 config9.m4 getopt.c php_getopt.h /sapi/cli config.w32 ge
janiMon Oct 1 12:32:39 2007 UTC Added files: /php-src/main getopt.c php_getopt.h Removed files: /php-src/sapi/cgi getopt.c php_getopt.h /php-src/sapi/cli getopt.c php_getopt.h Modified files: /php-srcconfigure.in /php-src/ext/standard basic_functions.c config.m4 /php-src/sapi/cgi cgi_main.c config.w32 config9.m4 /php-src/sapi/cli config.w32 php_cli.c /php-src/win32/buildconfig.w32 Log: - Added common getopt implementation to core. - Added long-option feature to getopt(). - Made getopt() available on win32 systems. Patch by: David Soria Parra <[EMAIL PROTECTED]> [DOC]: These changes will be available from 5.3+ http://cvs.php.net/viewvc.cgi/php-src/configure.in?r1=1.640&r2=1.641&diff_format=u Index: php-src/configure.in diff -u php-src/configure.in:1.640 php-src/configure.in:1.641 --- php-src/configure.in:1.640 Mon Aug 20 14:28:29 2007 +++ php-src/configure.inMon Oct 1 12:32:39 2007 @@ -1,4 +1,4 @@ -## $Id: configure.in,v 1.640 2007/08/20 14:28:29 jani Exp $ -*- autoconf -*- +## $Id: configure.in,v 1.641 2007/10/01 12:32:39 jani Exp $ -*- autoconf -*- dnl ## Process this file with autoconf to produce a configure script. divert(1) @@ -1054,7 +1054,7 @@ if test "$PHP_CLI" != "no"; then PHP_CLI_TARGET="\$(SAPI_CLI_PATH)" PHP_INSTALL_CLI_TARGET="install-cli" - PHP_ADD_SOURCES(sapi/cli, php_cli.c php_cli_readline.c getopt.c,, cli) + PHP_ADD_SOURCES(sapi/cli, php_cli.c php_cli_readline.c,, cli) PHP_INSTALLED_SAPIS="cli $PHP_SAPI" else PHP_INSTALLED_SAPIS="$PHP_SAPI" @@ -1208,7 +1208,7 @@ php_ini.c SAPI.c rfc1867.c php_content_types.c strlcpy.c \ strlcat.c mergesort.c reentrancy.c php_variables.c php_ticks.c \ network.c php_open_temporary_file.c php_logos.c \ - output.c ) + output.c getopt.c) PHP_ADD_SOURCES(main/streams, streams.c cast.c memory.c filter.c \ plain_wrapper.c userspace.c transports.c xp_socket.c mmap.c \ http://cvs.php.net/viewvc.cgi/php-src/ext/standard/basic_functions.c?r1=1.876&r2=1.877&diff_format=u Index: php-src/ext/standard/basic_functions.c diff -u php-src/ext/standard/basic_functions.c:1.876 php-src/ext/standard/basic_functions.c:1.877 --- php-src/ext/standard/basic_functions.c:1.876Fri Sep 28 13:06:22 2007 +++ php-src/ext/standard/basic_functions.c Mon Oct 1 12:32:39 2007 @@ -17,7 +17,7 @@ +--+ */ -/* $Id: basic_functions.c,v 1.876 2007/09/28 13:06:22 jani Exp $ */ +/* $Id: basic_functions.c,v 1.877 2007/10/01 12:32:39 jani Exp $ */ #include "php.h" #include "php_streams.h" @@ -28,6 +28,7 @@ #include "php_math.h" #include "php_http.h" #include "php_incomplete_class.h" +#include "php_getopt.h" #include "ext/standard/info.h" #include "ext/session/php_session.h" #include "zend_operators.h" @@ -83,10 +84,6 @@ # include #endif -#ifdef HARTMUT_0 -#include -#endif - #ifdef PHP_WIN32 # include "win32/unistd.h" #endif @@ -100,7 +97,6 @@ #include "SAPI.h" #include "php_ticks.h" - #ifdef ZTS PHPAPI int basic_globals_id; #else @@ -725,13 +721,11 @@ ZEND_END_ARG_INFO() #endif -#ifdef HAVE_GETOPT static ZEND_BEGIN_ARG_INFO_EX(arginfo_getopt, 0, 0, 1) ZEND_ARG_INFO(0, options) ZEND_ARG_INFO(0, opts) /* ARRAY_INFO(0, opts, 1) */ ZEND_END_ARG_INFO() -#endif static ZEND_BEGIN_ARG_INFO(arginfo_flush, 0) @@ -3128,7 +3122,7 @@ PHP_FE(iptcembed, arginfo_iptcembed) PHP_FE(getimagesize, arginfo_getimagesize) PHP_FE(image_type_to_mime_type, arginfo_image_type_to_mime_type) - PHP_FE(image_type_to_extension, arginfo_image_type_to_extension) + PHP_FE(image_type_to_extension, arginfo_image_type_to_extension) PHP_FE(phpinfo, arginfo_phpinfo) PHP_FE(phpversion, arginfo_phpversion) @@ -3178,7 +3172,7 @@ #endif #ifdef HAVE_STRFMON - PHP_FE(money_format, arginfo_money_format) + PHP_FE(money_format, arginfo_money_format) #endif PHP_FE(substr,
[PHP-CVS] cvs: php-src /main php_ini.c
janiMon Oct 1 12:26:56 2007 UTC Modified files: /php-src/main php_ini.c Log: - Fix warning http://cvs.php.net/viewvc.cgi/php-src/main/php_ini.c?r1=1.158&r2=1.159&diff_format=u Index: php-src/main/php_ini.c diff -u php-src/main/php_ini.c:1.158 php-src/main/php_ini.c:1.159 --- php-src/main/php_ini.c:1.158Fri Sep 28 10:23:14 2007 +++ php-src/main/php_ini.c Mon Oct 1 12:26:55 2007 @@ -16,7 +16,7 @@ +--+ */ -/* $Id: php_ini.c,v 1.158 2007/09/28 10:23:14 jani Exp $ */ +/* $Id: php_ini.c,v 1.159 2007/10/01 12:26:55 jani Exp $ */ #include "php.h" #include "ext/standard/info.h" @@ -721,7 +721,7 @@ */ PHPAPI void php_ini_activate_config(HashTable *source_hash, int modify_type, int stage TSRMLS_DC) { - char *str; + zstr str; zval *data; uint str_len; ulong num_index; @@ -732,7 +732,7 @@ zend_hash_move_forward(source_hash) ) { zend_hash_get_current_data(source_hash, (void **) &data); - zend_alter_ini_entry_ex(str, str_len, Z_STRVAL_P(data), Z_STRLEN_P(data), modify_type, stage, 0 TSRMLS_CC); + zend_alter_ini_entry_ex(str.s, str_len, Z_STRVAL_P(data), Z_STRLEN_P(data), modify_type, stage, 0 TSRMLS_CC); } } /* }}} */ -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_3) /ext/standard string.c /ext/standard/tests/strings implode1.phpt join_error.phpt join_variation2.phpt
janiMon Oct 1 12:05:41 2007 UTC Modified files: (Branch: PHP_5_3) /php-src/ext/standard string.c /php-src/ext/standard/tests/strings implode1.phpt join_error.phpt join_variation2.phpt Log: MFH: Fixed bug #42789 http://cvs.php.net/viewvc.cgi/php-src/ext/standard/string.c?r1=1.445.2.14.2.69.2.1&r2=1.445.2.14.2.69.2.2&diff_format=u Index: php-src/ext/standard/string.c diff -u php-src/ext/standard/string.c:1.445.2.14.2.69.2.1 php-src/ext/standard/string.c:1.445.2.14.2.69.2.2 --- php-src/ext/standard/string.c:1.445.2.14.2.69.2.1 Sun Sep 30 05:40:23 2007 +++ php-src/ext/standard/string.c Mon Oct 1 12:05:41 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: string.c,v 1.445.2.14.2.69.2.1 2007/09/30 05:40:23 jani Exp $ */ +/* $Id: string.c,v 1.445.2.14.2.69.2.2 2007/10/01 12:05:41 jani Exp $ */ /* Synced with php 3.0 revision 1.193 1999-06-16 [ssb] */ @@ -988,7 +988,7 @@ if (argc == 1) { if (Z_TYPE_PP(arg1) != IS_ARRAY) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Argument to implode must be an array"); + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Argument must be an array"); return; } @@ -1008,7 +1008,7 @@ convert_to_string_ex(arg1); delim = *arg1; } else { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad arguments"); + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid arguments passed"); return; } } http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/implode1.phpt?r1=1.1.2.4.2.1&r2=1.1.2.4.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/implode1.phpt http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_error.phpt?r1=1.1.4.3&r2=1.1.4.4&diff_format=u Index: php-src/ext/standard/tests/strings/join_error.phpt diff -u php-src/ext/standard/tests/strings/join_error.phpt:1.1.4.3 php-src/ext/standard/tests/strings/join_error.phpt:1.1.4.4 --- php-src/ext/standard/tests/strings/join_error.phpt:1.1.4.3 Sun Sep 30 14:37:33 2007 +++ php-src/ext/standard/tests/strings/join_error.phpt Mon Oct 1 12:05:41 2007 @@ -1,5 +1,5 @@ --TEST-- -Test join() function : error conditions(Bug#42789) +Test join() function: error conditions --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_variation2.phpt?r1=1.1.4.2&r2=1.1.4.3&diff_format=u Index: php-src/ext/standard/tests/strings/join_variation2.phpt diff -u php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.4.2 php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.4.3 --- php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.4.2 Sat Sep 29 16:52:54 2007 +++ php-src/ext/standard/tests/strings/join_variation2.phpt Mon Oct 1 12:05:41 2007 @@ -105,94 +105,94 @@ --- Testing join() by supplying different values for 'pieces' argument --- -- Iteration 1 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 2 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 3 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 4 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 5 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 6 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 7 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 8 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 9 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 10 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 11 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 12 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 13 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 14 -- -Warning: join(): Bad arguments in %s on
[PHP-CVS] cvs: php-src /ext/standard string.c /ext/standard/tests/strings implode1.phpt join_error.phpt join_variation2.phpt
janiMon Oct 1 12:04:09 2007 UTC Modified files: /php-src/ext/standard string.c /php-src/ext/standard/tests/strings implode1.phpt join_error.phpt join_variation2.phpt Log: - Fixed bug #42789 (join() warning messages are not proper & different return value on php5/6) # For consistency between 5/6 also made the return value be NULL like it # has been for ages when invalid stuff is passed. http://cvs.php.net/viewvc.cgi/php-src/ext/standard/string.c?r1=1.651&r2=1.652&diff_format=u Index: php-src/ext/standard/string.c diff -u php-src/ext/standard/string.c:1.651 php-src/ext/standard/string.c:1.652 --- php-src/ext/standard/string.c:1.651 Fri Sep 7 02:28:26 2007 +++ php-src/ext/standard/string.c Mon Oct 1 12:04:09 2007 @@ -18,7 +18,7 @@ +--+ */ -/* $Id: string.c,v 1.651 2007/09/07 02:28:26 iliaa Exp $ */ +/* $Id: string.c,v 1.652 2007/10/01 12:04:09 jani Exp $ */ /* Synced with php 3.0 revision 1.193 1999-06-16 [ssb] */ @@ -1265,8 +1265,8 @@ if (argc == 1) { if (Z_TYPE_PP(arg1) != IS_ARRAY) { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Argument to implode must be an array"); - RETURN_FALSE; + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Argument must be an array"); + return; } else { SEPARATE_ZVAL(arg1); arr = *arg1; @@ -1293,8 +1293,8 @@ } delim = *arg1; } else { - php_error_docref(NULL TSRMLS_CC, E_WARNING, "Bad arguments"); - RETURN_FALSE; + php_error_docref(NULL TSRMLS_CC, E_WARNING, "Invalid arguments passed"); + return; } } http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/implode1.phpt?r1=1.3&r2=1.4&diff_format=u Index: php-src/ext/standard/tests/strings/implode1.phpt http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_error.phpt?r1=1.1&r2=1.2&diff_format=u Index: php-src/ext/standard/tests/strings/join_error.phpt diff -u php-src/ext/standard/tests/strings/join_error.phpt:1.1 php-src/ext/standard/tests/strings/join_error.phpt:1.2 --- php-src/ext/standard/tests/strings/join_error.phpt:1.1 Sat Sep 29 09:08:24 2007 +++ php-src/ext/standard/tests/strings/join_error.phpt Mon Oct 1 12:04:09 2007 @@ -1,5 +1,5 @@ --TEST-- -Test join() function : error conditions(Bug#42789) +Test join() function: error conditions --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_variation2.phpt?r1=1.1&r2=1.2&diff_format=u Index: php-src/ext/standard/tests/strings/join_variation2.phpt diff -u php-src/ext/standard/tests/strings/join_variation2.phpt:1.1 php-src/ext/standard/tests/strings/join_variation2.phpt:1.2 --- php-src/ext/standard/tests/strings/join_variation2.phpt:1.1 Sat Sep 29 09:08:24 2007 +++ php-src/ext/standard/tests/strings/join_variation2.phpt Mon Oct 1 12:04:09 2007 @@ -102,95 +102,95 @@ --- Testing join() by supplying different values for 'pieces' argument --- -- Iteration 1 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 2 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 3 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 4 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 5 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 6 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 7 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 8 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 9 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 10 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 11 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 12 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Invalid arguments passed in %s on line %d NULL -- Iteration 13 -- -Warnin
[PHP-CVS] cvs: php-src /ext/standard/tests/strings moneyformat.phpt
janiMon Oct 1 12:01:28 2007 UTC Modified files: /php-src/ext/standard/tests/strings moneyformat.phpt Log: - Added unicode result http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/moneyformat.phpt?r1=1.3&r2=1.4&diff_format=u Index: php-src/ext/standard/tests/strings/moneyformat.phpt diff -u php-src/ext/standard/tests/strings/moneyformat.phpt:1.3 php-src/ext/standard/tests/strings/moneyformat.phpt:1.4 --- php-src/ext/standard/tests/strings/moneyformat.phpt:1.3 Wed Jun 6 17:47:15 2007 +++ php-src/ext/standard/tests/strings/moneyformat.phpt Mon Oct 1 12:01:28 2007 @@ -17,3 +17,9 @@ Warning: money_format(): Only a single %ci or %cn token can be used in %s on line %d bool(false) +--UEXPECTF-- +Strict Standards: setlocale(): deprecated in Unicode mode, please use ICU locale functions in /home/jani/src/php6/ext/standard/tests/strings/moneyformat.php on line 2 +unicode(7) "X$3.14Y" + +Warning: money_format(): Only a single %i or %n token can be used in /home/jani/src/php6/ext/standard/tests/strings/moneyformat.php on line 4 +bool(false) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/standard/tests/strings join_variation2.phpt
tony2001Mon Oct 1 10:41:36 2007 UTC Modified files: (Branch: PHP_5_2) /php-src/ext/standard/tests/strings join_variation2.phpt Log: fix test http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_variation2.phpt?r1=1.1.2.2&r2=1.1.2.3&diff_format=u Index: php-src/ext/standard/tests/strings/join_variation2.phpt diff -u php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.2.2 php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.2.3 --- php-src/ext/standard/tests/strings/join_variation2.phpt:1.1.2.2 Sat Sep 29 09:10:18 2007 +++ php-src/ext/standard/tests/strings/join_variation2.phpt Mon Oct 1 10:41:36 2007 @@ -105,94 +105,94 @@ --- Testing join() by supplying different values for 'pieces' argument --- -- Iteration 1 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 2 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 3 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 4 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 5 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 6 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 7 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 8 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 9 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 10 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 11 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 12 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 13 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 14 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 15 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 16 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 17 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 18 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 19 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 20 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 21 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 22 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL -- Iteration 23 -- -Warning: join(): Bad arguments in %s on line %d +Warning: join(): Bad arguments. in %s on line %d NULL Done -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/standard/tests/strings join_error.phpt sprintf_basic7.phpt sprintf_basic8.phpt sprintf_variation28.phpt sprintf_variation29.phpt sprintf_variation34.phpt sprintf_v
tony2001Mon Oct 1 10:25:50 2007 UTC Modified files: (Branch: PHP_5_2) /php-src/ext/standard/tests/strings join_error.phpt sprintf_basic7.phpt sprintf_basic8.phpt sprintf_variation28.phpt sprintf_variation29.phpt sprintf_variation34.phpt sprintf_variation35.phpt sprintf_variation4.phpt sprintf_variation40.phpt sprintf_variation41.phpt sprintf_variation44.phpt Log: fix tests http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/join_error.phpt?r1=1.1.2.2&r2=1.1.2.3&diff_format=u Index: php-src/ext/standard/tests/strings/join_error.phpt diff -u php-src/ext/standard/tests/strings/join_error.phpt:1.1.2.2 php-src/ext/standard/tests/strings/join_error.phpt:1.1.2.3 --- php-src/ext/standard/tests/strings/join_error.phpt:1.1.2.2 Sat Sep 29 09:10:18 2007 +++ php-src/ext/standard/tests/strings/join_error.phpt Mon Oct 1 10:25:50 2007 @@ -45,6 +45,6 @@ -- Testing join() with less than expected no. of arguments -- -Warning: join(): Argument to join must be an array in %s on line %d +Warning: join(): Argument to implode must be an array. in %s on line %d NULL Done http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_basic7.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_basic7.phpt diff -u php-src/ext/standard/tests/strings/sprintf_basic7.phpt:1.1.2.1 php-src/ext/standard/tests/strings/sprintf_basic7.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/sprintf_basic7.phpt:1.1.2.1 Sat Sep 29 14:13:53 2007 +++ php-src/ext/standard/tests/strings/sprintf_basic7.phpt Mon Oct 1 10:25:50 2007 @@ -1,5 +1,11 @@ --TEST-- Test sprintf() function : basic functionality - unsigned format +--SKIPIF-- + --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_basic8.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_basic8.phpt diff -u php-src/ext/standard/tests/strings/sprintf_basic8.phpt:1.1.2.1 php-src/ext/standard/tests/strings/sprintf_basic8.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/sprintf_basic8.phpt:1.1.2.1 Sat Sep 29 14:13:53 2007 +++ php-src/ext/standard/tests/strings/sprintf_basic8.phpt Mon Oct 1 10:25:50 2007 @@ -1,5 +1,11 @@ --TEST-- Test sprintf() function : basic functionality - octal format +--SKIPIF-- + --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_variation28.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_variation28.phpt diff -u php-src/ext/standard/tests/strings/sprintf_variation28.phpt:1.1.2.1 php-src/ext/standard/tests/strings/sprintf_variation28.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/sprintf_variation28.phpt:1.1.2.1 Sat Sep 29 14:13:53 2007 +++ php-src/ext/standard/tests/strings/sprintf_variation28.phpt Mon Oct 1 10:25:50 2007 @@ -1,5 +1,11 @@ --TEST-- Test sprintf() function : usage variations - octal formats with integer values +--SKIPIF-- + --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_variation29.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_variation29.phpt diff -u php-src/ext/standard/tests/strings/sprintf_variation29.phpt:1.1.2.1 php-src/ext/standard/tests/strings/sprintf_variation29.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/sprintf_variation29.phpt:1.1.2.1 Sat Sep 29 14:13:53 2007 +++ php-src/ext/standard/tests/strings/sprintf_variation29.phpt Mon Oct 1 10:25:50 2007 @@ -1,5 +1,11 @@ --TEST-- Test sprintf() function : usage variations - octal formats with float values +--SKIPIF-- + --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_variation34.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_variation34.phpt diff -u php-src/ext/standard/tests/strings/sprintf_variation34.phpt:1.1.2.1 php-src/ext/standard/tests/strings/sprintf_variation34.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/sprintf_variation34.phpt:1.1.2.1 Sat Sep 29 14:13:53 2007 +++ php-src/ext/standard/tests/strings/sprintf_variation34.phpt Mon Oct 1 10:25:50 2007 @@ -1,5 +1,11 @@ --TEST-- Test sprintf() function : usage variations - hexa formats with integer values +--SKIPIF-- + --FILE-- http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/sprintf_variation35.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/sprintf_variation35.phpt diff -u php-src/ext/standard/tests/strings/spri
[PHP-CVS] cvs: php-src(PHP_5_2) /ext/standard/tests/strings chunk_split_variation2.phpt chunk_split_variation5.phpt chunk_split_variation8.phpt
tony2001Mon Oct 1 10:22:43 2007 UTC Modified files: (Branch: PHP_5_2) /php-src/ext/standard/tests/strings chunk_split_variation2.phpt chunk_split_variation5.phpt chunk_split_variation8.phpt Log: these dots exist since PHP3 even though Jani removed them in 5_3, I can't see how they can cause any BUGS http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/chunk_split_variation2.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/chunk_split_variation2.phpt diff -u php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.1 php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.2 --- php-src/ext/standard/tests/strings/chunk_split_variation2.phpt:1.1.2.1 Sat Sep 29 12:55:34 2007 +++ php-src/ext/standard/tests/strings/chunk_split_variation2.phpt Mon Oct 1 10:22:43 2007 @@ -95,21 +95,21 @@ string(28) "This is ch*uklen vari*ation*" -- Iteration 2 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 3 -- string(26) "This is chuklen variation*" -- Iteration 4 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 5 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 6 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 7 -- string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" @@ -121,39 +121,39 @@ string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" -- Iteration 11 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 12 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 13 -- string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" -- Iteration 14 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 15 -- string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" -- Iteration 16 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 17 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 18 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 19 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 20 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 21 -- @@ -161,11 +161,11 @@ string(50) "T*h*i*s* *i*s* *c*h*u*k*l*e*n* *v*a*r*i*a*t*i*o*n*" -- Iteration 22 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 23 -- -Warning: chunk_split(): Chunk length should be greater than zero in %s on line %d +Warning: chunk_split(): Chunk length should be greater than zero. in %s on line %d bool(false) -- Iteration 24 -- string(30) "This *is ch*uklen* vari*ation*" http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/chunk_split_variation5.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/chunk_split_variation5.phpt http://cvs.php.net/viewvc.cgi/php-src/ext/standard/tests/strings/chunk_split_variation8.phpt?r1=1.1.2.1&r2=1.1.2.2&diff_format=u Index: php-src/ext/standard/tests/strings/chunk_split_variation8.phpt diff -u php-src/ext/standard/tests/strings/chunk_split_variation8.phpt:1.1.2.1 php-src/ext/standard/tests/strings/chunk_spl
[PHP-CVS] cvs: php-src(PHP_5_2) / README.UPDATE_5_2
pajoye Mon Oct 1 08:14:03 2007 UTC Modified files: (Branch: PHP_5_2) /php-srcREADME.UPDATE_5_2 Log: - add notice about glob regression (Lukas) http://cvs.php.net/viewvc.cgi/php-src/README.UPDATE_5_2?r1=1.1.2.39&r2=1.1.2.40&diff_format=u Index: php-src/README.UPDATE_5_2 diff -u php-src/README.UPDATE_5_2:1.1.2.39 php-src/README.UPDATE_5_2:1.1.2.40 --- php-src/README.UPDATE_5_2:1.1.2.39 Sat Jan 6 18:05:02 2007 +++ php-src/README.UPDATE_5_2 Mon Oct 1 08:14:03 2007 @@ -172,6 +172,21 @@ from your error handler in order to populate $php_errormsg. This allows you to fine-grain the levels of the messages stored. +== +Regressions introduced/fixed +== + +- In version 5.2.4 a security fix caused a regression for patterns of + the form "/foo/*/bar/*". Since version 5.2.5 instead of raising a warning the + glob() function will return false when openbase_dir restrictions are violated. + + #See http://bugs.php.net/bug.php?id=41655 + + The warning that used to be raised looked like something as follows: + Warning: glob() [function.glob]: Unable to access /foo/*/bar/* in /foo.php on line xxx + #See http://cvs.php.net/viewvc.cgi/php-src/ext/standard/dir.c?r1=1.169&r2=1.170 + + The patch can safely be applied to PHP 5.2.4 as well. == Backwards incompatible changes -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php