[PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ NEWS
mariuz Tue, 27 Dec 2011 11:05:58 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321434
Log:
Forgot to add the Pdo Firebird related bug to the News
Changed paths:
U php/php-src/branches/PHP_5_3/NEWS
Modified: php/php-src/branches/PHP_5_3/NEWS
===
--- php/php-src/branches/PHP_5_3/NEWS 2011-12-27 10:54:47 UTC (rev 321433)
+++ php/php-src/branches/PHP_5_3/NEWS 2011-12-27 11:05:58 UTC (rev 321434)
@@ -15,6 +15,11 @@
. Fixed bug #60455 (stream_get_line misbehaves if EOF is not detected
together
with the last read). (Gustavo)
+- Pdo Firebird:
+ . Fixed bug #48877 ("bindValue" and "bindParam" do not work for PDO
Firebird).
+(Mariuz)
+
+
08 Dec 2011, PHP 5.3.9RC3
- Filter:
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/pdo_firebird/tests/ bug_48877.phpt
mariuz Tue, 27 Dec 2011 10:49:03 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321432
Log:
added test for PDO_Firebird: bug 48877
Bug: https://bugs.php.net/48877 (Closed) "bindValue" and "bindParam" do not
work for PDO Firebird
Changed paths:
A php/php-src/branches/PHP_5_4/ext/pdo_firebird/tests/bug_48877.phpt
Added: php/php-src/branches/PHP_5_4/ext/pdo_firebird/tests/bug_48877.phpt
===
--- php/php-src/branches/PHP_5_4/ext/pdo_firebird/tests/bug_48877.phpt
(rev 0)
+++ php/php-src/branches/PHP_5_4/ext/pdo_firebird/tests/bug_48877.phpt
2011-12-27 10:49:03 UTC (rev 321432)
@@ -0,0 +1,42 @@
+--TEST--
+PDO_Firebird: bug 48877 test
+--SKIPIF--
+
+--FILE--
+exec('DROP TABLE testz');
+$dbh->exec('CREATE TABLE testz (A integer)');
+$dbh->exec("INSERT INTO testz VALUES ('1')");
+$dbh->exec("INSERT INTO testz VALUES ('2')");
+$dbh->exec("INSERT INTO testz VALUES ('3')");
+$dbh->commit();
+
+$query = "SELECT * FROM testz WHERE A = :paramno";
+
+$stmt = $dbh->prepare($query);
+$stmt->bindParam(':paramno', $value, PDO::PARAM_STR);
+$stmt->execute();
+$rows = $stmt->fetch();
+var_dump($stmt->fetch());
+var_dump($stmt->rowCount());
+
+
+$stmt = $dbh->prepare('DELETE FROM testz');
+$stmt->execute();
+
+$dbh->commit();
+
+$dbh->exec('DROP TABLE testz');
+
+unset($stmt);
+unset($dbh);
+
+?>
+--EXPECT--
+bool(false)
+int(1)
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/date/tests/ DateTime_add-fall-type2-type2.phpt
On Tue, 27 Dec 2011, Daniel Convissor wrote: > > Why are you only changing this in the 5.4 branch? > > There are still more changes to be made and I'll merge them all to > trunk and 5.3. Please make those changes in *one* commit in the future. Right now there are inconsistencies between tests (again). cheers, Derick -- http://derickrethans.nl | http://xdebug.org Like Xdebug? Consider a donation: http://xdebug.org/donate.php twitter: @derickr and @xdebug -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/date/tests/ DateTime_add-fall-type2-type2.phpt
On Tue, 27 Dec 2011, Daniel Convissor wrote: > danielc Tue, 27 Dec 2011 02:13:55 + > > Revision: http://svn.php.net/viewvc?view=revision&revision=321419 > > Log: > One less xfail since Bug 55253 was fixed. > > Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 > hour on objects with time zone type 2 > > Changed paths: > U > php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_add-fall-type2-type2.phpt Why are you only changing this in the 5.4 branch? Derick -- http://derickrethans.nl | http://xdebug.org Like Xdebug? Consider a donation: http://xdebug.org/donate.php twitter: @derickr and @xdebug -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/pdo_firebird/firebird_driver.c branches/PHP_5_4/ext/pdo_firebird/firebird_driver.c trunk/ext/pdo_firebird/firebird_driver.c
mariuz Tue, 27 Dec 2011 16:49:00 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321439
Log:
http://gcov.php.net/viewer.php?version=PHP_5_3&func=tests&file=ext%2Fpdo_firebird%2Ftests%2Fbug_43130.phpt
fix failure for bug 43130
Bug: https://bugs.php.net/43130 (Wont fix) bind parameter cannot contain dashes
Changed paths:
U php/php-src/branches/PHP_5_3/ext/pdo_firebird/firebird_driver.c
U php/php-src/branches/PHP_5_4/ext/pdo_firebird/firebird_driver.c
U php/php-src/trunk/ext/pdo_firebird/firebird_driver.c
Modified: php/php-src/branches/PHP_5_3/ext/pdo_firebird/firebird_driver.c
===
--- php/php-src/branches/PHP_5_3/ext/pdo_firebird/firebird_driver.c
2011-12-27 16:38:23 UTC (rev 321438)
+++ php/php-src/branches/PHP_5_3/ext/pdo_firebird/firebird_driver.c
2011-12-27 16:49:00 UTC (rev 321439)
@@ -440,8 +440,10 @@
continue;
}
} else {
- if ((in_param &= (sql[l] == '_') || (sql[l] >=
'A' && sql[l] <= 'Z')
- || (sql[l] >= 'a' && sql[l] <=
'z') || (sql[l] >= '0' && sql[l] <= '9'))) {
+if ((in_param &= ((sql[l] >= 'A' && sql[l] <=
'Z') || (sql[l] >= 'a' && sql[l] <= 'z')
+|| (sql[l] >= '0' && sql[l] <= '9') ||
sql[l] == '_' || sql[l] == '-'))) {
+
+
*ppname++ = sql[l];
continue;
} else {
Modified: php/php-src/branches/PHP_5_4/ext/pdo_firebird/firebird_driver.c
===
--- php/php-src/branches/PHP_5_4/ext/pdo_firebird/firebird_driver.c
2011-12-27 16:38:23 UTC (rev 321438)
+++ php/php-src/branches/PHP_5_4/ext/pdo_firebird/firebird_driver.c
2011-12-27 16:49:00 UTC (rev 321439)
@@ -440,8 +440,10 @@
continue;
}
} else {
- if ((in_param &= (sql[l] == '_') || (sql[l] >=
'A' && sql[l] <= 'Z')
- || (sql[l] >= 'a' && sql[l] <=
'z') || (sql[l] >= '0' && sql[l] <= '9'))) {
+if ((in_param &= ((sql[l] >= 'A' && sql[l] <=
'Z') || (sql[l] >= 'a' && sql[l] <= 'z')
+|| (sql[l] >= '0' && sql[l] <= '9') ||
sql[l] == '_' || sql[l] == '-'))) {
+
+
*ppname++ = sql[l];
continue;
} else {
Modified: php/php-src/trunk/ext/pdo_firebird/firebird_driver.c
===
--- php/php-src/trunk/ext/pdo_firebird/firebird_driver.c2011-12-27
16:38:23 UTC (rev 321438)
+++ php/php-src/trunk/ext/pdo_firebird/firebird_driver.c2011-12-27
16:49:00 UTC (rev 321439)
@@ -440,8 +440,10 @@
continue;
}
} else {
- if ((in_param &= (sql[l] == '_') || (sql[l] >=
'A' && sql[l] <= 'Z')
- || (sql[l] >= 'a' && sql[l] <=
'z') || (sql[l] >= '0' && sql[l] <= '9'))) {
+if ((in_param &= ((sql[l] >= 'A' && sql[l] <=
'Z') || (sql[l] >= 'a' && sql[l] <= 'z')
+|| (sql[l] >= '0' && sql[l] <= '9') ||
sql[l] == '_' || sql[l] == '-'))) {
+
+
*ppname++ = sql[l];
continue;
} else {
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/date/tests/ DateTime_add-spring-type2-type2.phpt
danielc Tue, 27 Dec 2011 16:50:16 + Revision: http://svn.php.net/viewvc?view=revision&revision=321441 Log: Another xfail resolved since Bug 55253 was fixed. Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 hour on objects with time zone type 2 Changed paths: U php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_add-spring-type2-type2.phpt Modified: php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_add-spring-type2-type2.phpt === --- php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_add-spring-type2-type2.phpt 2011-12-27 16:49:34 UTC (rev 321440) +++ php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_add-spring-type2-type2.phpt 2011-12-27 16:50:16 UTC (rev 321441) @@ -2,8 +2,6 @@ DateTime::add() -- spring type2 type2 --CREDITS-- Daniel Convissor ---XFAIL-- -Various bugs exist --FILE-- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/date/tests/DateTime_add-fall-type2-type2.phpt branches/PHP_5_3/ext/date/tests/DateTime_add-spring-type2-type2.phpt branches/PHP_5_3/ext/date/tests/Dat
danielc Tue, 27 Dec 2011 16:57:36 + Revision: http://svn.php.net/viewvc?view=revision&revision=321442 Log: Resolve some xfails due to Bug 55253 being fixed. (merged from 54) Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 hour on objects with time zone type 2 Changed paths: U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_add-fall-type2-type2.phpt U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_add-spring-type2-type2.phpt U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_sub-fall-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_add-fall-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_add-spring-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_sub-fall-type2-type2.phpt Modified: php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_add-fall-type2-type2.phpt === --- php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_add-fall-type2-type2.phpt 2011-12-27 16:50:16 UTC (rev 321441) +++ php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_add-fall-type2-type2.phpt 2011-12-27 16:57:36 UTC (rev 321442) @@ -2,8 +2,6 @@ DateTime::add() -- fall type2 type2 --CREDITS-- Daniel Convissor ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/acinclude.m4 trunk/acinclude.m4
bjoriTue, 27 Dec 2011 13:53:11 + Revision: http://svn.php.net/viewvc?view=revision&revision=321435 Log: Looks like we need to explicity add libstdc++ on recent linux' too, like fedora 14 and ubuntu 11 This fixes build failures for ext/intl and several pecl exts Changed paths: U php/php-src/branches/PHP_5_4/NEWS U php/php-src/branches/PHP_5_4/acinclude.m4 U php/php-src/trunk/acinclude.m4 Modified: php/php-src/branches/PHP_5_4/NEWS === --- php/php-src/branches/PHP_5_4/NEWS 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/branches/PHP_5_4/NEWS 2011-12-27 13:53:11 UTC (rev 321435) @@ -7,6 +7,10 @@ - CLI SAPI: . Fixed bug #60591 (Memory leak when access a non-exists file). (Laruence) +- Intl: + . Fixed build on Fedora 15 / Ubuntu 11. (Hannes) + + 22 Dec 2011, PHP 5.4.0 RC4 - Core: . Added max_input_vars directive to prevent attacks based on hash collisions Modified: php/php-src/branches/PHP_5_4/acinclude.m4 === --- php/php-src/branches/PHP_5_4/acinclude.m4 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/branches/PHP_5_4/acinclude.m4 2011-12-27 13:53:11 UTC (rev 321435) @@ -762,11 +762,7 @@ if test -z "$php_cxx_done"; then AC_PROG_CXX AC_PROG_CXXCPP -case $host_alias in - *darwin*) -PHP_ADD_LIBRARY(stdc++) - ;; -esac +PHP_ADD_LIBRARY(stdc++) php_cxx_done=yes fi ]) Modified: php/php-src/trunk/acinclude.m4 === --- php/php-src/trunk/acinclude.m4 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/trunk/acinclude.m4 2011-12-27 13:53:11 UTC (rev 321435) @@ -762,11 +762,7 @@ if test -z "$php_cxx_done"; then AC_PROG_CXX AC_PROG_CXXCPP -case $host_alias in - *darwin*) -PHP_ADD_LIBRARY(stdc++) - ;; -esac +PHP_ADD_LIBRARY(stdc++) php_cxx_done=yes fi ]) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/date/tests/ DateTime_add-fall-type2-type2.phpt
Hi Derick: > Why are you only changing this in the 5.4 branch? There are still more changes to be made and I'll merge them all to trunk and 5.3. Thanks, --Dan -- T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y data intensive web and database programming http://www.AnalysisAndSolutions.com/ 4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409 -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/acinclude.m4 trunk/acinclude.m4
Uhm, I don't think this change makes much sense (nor does the previous behaviour on Mac).. Extensions that use C++ should pull in the required libraries themselves. We shouldn't be linking PHP everytime with libstdc++ if that is not needed. Nuno - Original Message - bjoriTue, 27 Dec 2011 13:53:11 + Revision: http://svn.php.net/viewvc?view=revision&revision=321435 Log: Looks like we need to explicity add libstdc++ on recent linux' too, like fedora 14 and ubuntu 11 This fixes build failures for ext/intl and several pecl exts Changed paths: U php/php-src/branches/PHP_5_4/NEWS U php/php-src/branches/PHP_5_4/acinclude.m4 U php/php-src/trunk/acinclude.m4 Modified: php/php-src/branches/PHP_5_4/NEWS === --- php/php-src/branches/PHP_5_4/NEWS 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/branches/PHP_5_4/NEWS 2011-12-27 13:53:11 UTC (rev 321435) @@ -7,6 +7,10 @@ - CLI SAPI: . Fixed bug #60591 (Memory leak when access a non-exists file). (Laruence) +- Intl: + . Fixed build on Fedora 15 / Ubuntu 11. (Hannes) + + 22 Dec 2011, PHP 5.4.0 RC4 - Core: . Added max_input_vars directive to prevent attacks based on hash collisions Modified: php/php-src/branches/PHP_5_4/acinclude.m4 === --- php/php-src/branches/PHP_5_4/acinclude.m4 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/branches/PHP_5_4/acinclude.m4 2011-12-27 13:53:11 UTC (rev 321435) @@ -762,11 +762,7 @@ if test -z "$php_cxx_done"; then AC_PROG_CXX AC_PROG_CXXCPP -case $host_alias in - *darwin*) -PHP_ADD_LIBRARY(stdc++) - ;; -esac +PHP_ADD_LIBRARY(stdc++) php_cxx_done=yes fi ]) Modified: php/php-src/trunk/acinclude.m4 === --- php/php-src/trunk/acinclude.m4 2011-12-27 11:05:58 UTC (rev 321434) +++ php/php-src/trunk/acinclude.m4 2011-12-27 13:53:11 UTC (rev 321435) @@ -762,11 +762,7 @@ if test -z "$php_cxx_done"; then AC_PROG_CXX AC_PROG_CXXCPP -case $host_alias in - *darwin*) -PHP_ADD_LIBRARY(stdc++) - ;; -esac +PHP_ADD_LIBRARY(stdc++) php_cxx_done=yes fi ]) -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/Zend/tests/bug60611.phpt branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/tests/bug60611.phpt trunk/Zend/zend_compile.c
laruence Tue, 27 Dec 2011 08:38:18 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321429
Log:
Fix bug #60611 (Segmentation fault with Cls::{expr}() syntax)
Bug: https://bugs.php.net/60611 (Open) Segmentation fault with Cls::{expr}()
syntax
Changed paths:
U php/php-src/branches/PHP_5_4/NEWS
A php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt
U php/php-src/branches/PHP_5_4/Zend/zend_compile.c
A php/php-src/trunk/Zend/tests/bug60611.phpt
U php/php-src/trunk/Zend/zend_compile.c
Modified: php/php-src/branches/PHP_5_4/NEWS
===
--- php/php-src/branches/PHP_5_4/NEWS 2011-12-27 06:24:33 UTC (rev 321428)
+++ php/php-src/branches/PHP_5_4/NEWS 2011-12-27 08:38:18 UTC (rev 321429)
@@ -1,6 +1,9 @@
PHPNEWS
|||
?? Jan 2012, PHP 5.4.0 RC5
+- Core:
+ . Fixed bug #60611 (Segmentation fault with Cls::{expr}() syntax). (Laruence)
+
- CLI SAPI:
. Fixed bug #60591 (Memory leak when access a non-exists file). (Laruence)
Added: php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt
===
--- php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt
(rev 0)
+++ php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt 2011-12-27
08:38:18 UTC (rev 321429)
@@ -0,0 +1,28 @@
+--TEST--
+Bug #60611 (Segmentation fault with Cls::{expr}() syntax)
+--FILE--
+{0}();
+$cls->{1.0}();
+$cls->{true}();
+$cls->{false}();
+$cls->{null}();
+
+echo "done";
+?>
+--EXPECT--
+done
Modified: php/php-src/branches/PHP_5_4/Zend/zend_compile.c
===
--- php/php-src/branches/PHP_5_4/Zend/zend_compile.c2011-12-27 06:24:33 UTC
(rev 321428)
+++ php/php-src/branches/PHP_5_4/Zend/zend_compile.c2011-12-27 08:38:18 UTC
(rev 321429)
@@ -1973,9 +1973,10 @@
if (last_op->opcode == ZEND_FETCH_OBJ_R) {
if (last_op->op2_type == IS_CONST) {
zval name;
-
name = CONSTANT(last_op->op2.constant);
- if (!IS_INTERNED(Z_STRVAL(name))) {
+ if (Z_TYPE(name) != IS_STRING) {
+ convert_to_string(&name);
+ } else if (!IS_INTERNED(Z_STRVAL(name))) {
Z_STRVAL(name) = estrndup(Z_STRVAL(name),
Z_STRLEN(name));
}
FREE_POLYMORPHIC_CACHE_SLOT(last_op->op2.constant);
@@ -2367,7 +2368,11 @@
zend_op *opline;
if (method_name->op_type == IS_CONST) {
- char *lcname =
zend_str_tolower_dup(Z_STRVAL(method_name->u.constant),
Z_STRLEN(method_name->u.constant));
+ char *lcname;
+ if (Z_TYPE(method_name->u.constant) != IS_STRING) {
+ convert_to_string(&method_name->u.constant);
+ }
+ lcname =
zend_str_tolower_dup(Z_STRVAL(method_name->u.constant),
Z_STRLEN(method_name->u.constant));
if ((sizeof(ZEND_CONSTRUCTOR_FUNC_NAME)-1) ==
Z_STRLEN(method_name->u.constant) &&
memcmp(lcname, ZEND_CONSTRUCTOR_FUNC_NAME,
sizeof(ZEND_CONSTRUCTOR_FUNC_NAME)-1) == 0) {
zval_dtor(&method_name->u.constant);
Added: php/php-src/trunk/Zend/tests/bug60611.phpt
===
--- php/php-src/trunk/Zend/tests/bug60611.phpt (rev 0)
+++ php/php-src/trunk/Zend/tests/bug60611.phpt 2011-12-27 08:38:18 UTC (rev
321429)
@@ -0,0 +1,28 @@
+--TEST--
+Bug #60611 (Segmentation fault with Cls::{expr}() syntax)
+--FILE--
+{0}();
+$cls->{1.0}();
+$cls->{true}();
+$cls->{false}();
+$cls->{null}();
+
+echo "done";
+?>
+--EXPECT--
+done
Modified: php/php-src/trunk/Zend/zend_compile.c
===
--- php/php-src/trunk/Zend/zend_compile.c 2011-12-27 06:24:33 UTC (rev
321428)
+++ php/php-src/trunk/Zend/zend_compile.c 2011-12-27 08:38:18 UTC (rev
321429)
@@ -1973,9 +1973,10 @@
if (last_op->opcode == ZEND_FETCH_OBJ_R) {
if (last_op->op2_type == IS_CONST) {
zval name;
-
name = CONSTANT(last_op->op2.constant);
- if (!IS_INTERNED(Z_STRVAL(name))) {
+ if (Z_TYPE(name) != IS_STRING) {
+ convert_to_string(&name);
+ } else if (!IS_INTERNED(Z_STRVAL(name))) {
Z_STRVAL(name) = estrndup(Z_STRVAL(name),
Z_STRLEN(name));
}
FREE_POLYMORPHIC_CACHE_SLOT(last_op->op2.constant);
@@
Re: [PHP-CVS] svn: /php/php-src/branches/PHP_5_3/ext/pdo_firebird/ firebird_driver.c
On Tue, Dec 13, 2011 at 11:22 PM, Pierre Joye wrote: > do a sparse checkout and commit all branches at once from php-src, see > https://wiki.php.net/vcs/svnfaq :) > > 2011/12/13 marius adrian popa : >> 2011/12/13 Johannes Schlüter : >>> Hi. >>> >>> Is there a test? >> I will add one to the tree >>> >>> And btw. Usually we commit to all branches at once, which makes it >>> simpler to see if a patch is applied to all branches. >> I will try to do that in the future , any tip ? I have added the testcase , but no sparse yet Thanks i will try to commit my next svn patches using the sparse method -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/trunk/ext/pdo_firebird/tests/ bug_48877.phpt
mariuz Tue, 27 Dec 2011 10:54:47 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321433
Log:
PDO_Firebird: bug 48877 test
Bug: https://bugs.php.net/48877 (Closed) "bindValue" and "bindParam" do not
work for PDO Firebird
Changed paths:
A php/php-src/trunk/ext/pdo_firebird/tests/bug_48877.phpt
Added: php/php-src/trunk/ext/pdo_firebird/tests/bug_48877.phpt
===
--- php/php-src/trunk/ext/pdo_firebird/tests/bug_48877.phpt
(rev 0)
+++ php/php-src/trunk/ext/pdo_firebird/tests/bug_48877.phpt 2011-12-27
10:54:47 UTC (rev 321433)
@@ -0,0 +1,42 @@
+--TEST--
+PDO_Firebird: bug 48877 test
+--SKIPIF--
+
+--FILE--
+exec('DROP TABLE testz');
+$dbh->exec('CREATE TABLE testz (A integer)');
+$dbh->exec("INSERT INTO testz VALUES ('1')");
+$dbh->exec("INSERT INTO testz VALUES ('2')");
+$dbh->exec("INSERT INTO testz VALUES ('3')");
+$dbh->commit();
+
+$query = "SELECT * FROM testz WHERE A = :paramno";
+
+$stmt = $dbh->prepare($query);
+$stmt->bindParam(':paramno', $value, PDO::PARAM_STR);
+$stmt->execute();
+$rows = $stmt->fetch();
+var_dump($stmt->fetch());
+var_dump($stmt->rowCount());
+
+
+$stmt = $dbh->prepare('DELETE FROM testz');
+$stmt->execute();
+
+$dbh->commit();
+
+$dbh->exec('DROP TABLE testz');
+
+unset($stmt);
+unset($dbh);
+
+?>
+--EXPECT--
+bool(false)
+int(1)
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/date/tests/DateTime_sub-spring-type2-type2.phpt branches/PHP_5_4/ext/date/tests/DateTime_sub-spring-type2-type2.phpt trunk/ext/date/tests/DateTime_sub
danielc Tue, 27 Dec 2011 19:44:13 + Revision: http://svn.php.net/viewvc?view=revision&revision=321447 Log: Resolve more xfails due to Bug 55253 being fixed. Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 hour on objects with time zone type 2 Changed paths: U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_sub-spring-type2-type2.phpt U php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_sub-spring-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_sub-spring-type2-type2.phpt Modified: php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_sub-spring-type2-type2.phpt === --- php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_sub-spring-type2-type2.phpt 2011-12-27 19:32:56 UTC (rev 321446) +++ php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_sub-spring-type2-type2.phpt 2011-12-27 19:44:13 UTC (rev 321447) @@ -2,8 +2,6 @@ DateTime::sub() -- spring type2 type2 --CREDITS-- Daniel Convissor ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/acinclude.m4 trunk/acinclude.m4
On Tue, Dec 27, 2011 at 18:02, Nuno Lopes wrote: > Uhm, I don't think this change makes much sense (nor does the previous > behaviour on Mac).. > Extensions that use C++ should pull in the required libraries themselves. We > shouldn't be linking PHP everytime with libstdc++ if that is not needed. Then what is the idea behind PHP_REQUIRE_CXX? As I see it, it should pull in everything needed for the extension to use c++. -Hannes -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
Re: [PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/NEWS branches/PHP_5_4/acinclude.m4 trunk/acinclude.m4
On Tue, Dec 27, 2011 at 18:02, Nuno Lopes wrote: Uhm, I don't think this change makes much sense (nor does the previous behaviour on Mac).. Extensions that use C++ should pull in the required libraries themselves. We shouldn't be linking PHP everytime with libstdc++ if that is not needed. Then what is the idea behind PHP_REQUIRE_CXX? As I see it, it should pull in everything needed for the extension to use c++. Ah, sorry, I was fooled by your commit message.. I didn't noticed your change was in PHP_REQUIRE_CXX. Anyway, pulling libstdc++ is a bit too gcc centric. If I use, say, clang, then I may want to use libc++ instead. The right fix, I believe, is to change the link command to use $CXX instead of $CC, since the c++ compiler knows which libraries it needs. For example, it seems that PHP_SHARED_MODULE is doing the right thing. Nuno -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_3/ext/date/tests/DateTime_diff-fall-type2-type2.phpt branches/PHP_5_3/ext/date/tests/DateTime_diff-spring-type2-type2.phpt branches/PHP_5_4/ext/date/tests/D
danielc Wed, 28 Dec 2011 01:13:12 + Revision: http://svn.php.net/viewvc?view=revision&revision=321456 Log: Resolve the last of the xfails from Bug 55253. Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 hour on objects with time zone type 2 Changed paths: U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_diff-fall-type2-type2.phpt U php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_diff-spring-type2-type2.phpt U php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_diff-fall-type2-type2.phpt U php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_diff-spring-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_diff-fall-type2-type2.phpt U php/php-src/trunk/ext/date/tests/DateTime_diff-spring-type2-type2.phpt Modified: php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_diff-fall-type2-type2.phpt === --- php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_diff-fall-type2-type2.phpt 2011-12-27 23:44:03 UTC (rev 321455) +++ php/php-src/branches/PHP_5_3/ext/date/tests/DateTime_diff-fall-type2-type2.phpt 2011-12-28 01:13:12 UTC (rev 321456) @@ -2,8 +2,6 @@ DateTime::diff() -- fall type2 type2 --CREDITS-- Daniel Convissor ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- ---XFAIL-- -Various bugs exist --FILE-- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/branches/PHP_5_4/ext/date/tests/ DateTime_sub-fall-type2-type2.phpt
danielc Tue, 27 Dec 2011 15:20:16 + Revision: http://svn.php.net/viewvc?view=revision&revision=321437 Log: Another xfail resolved since Bug 55253 was fixed. Bug: https://bugs.php.net/55253 (Closed) DateTime::add() and sub() result -1 hour on objects with time zone type 2 Changed paths: U php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_sub-fall-type2-type2.phpt Modified: php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_sub-fall-type2-type2.phpt === --- php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_sub-fall-type2-type2.phpt 2011-12-27 14:29:32 UTC (rev 321436) +++ php/php-src/branches/PHP_5_4/ext/date/tests/DateTime_sub-fall-type2-type2.phpt 2011-12-27 15:20:16 UTC (rev 321437) @@ -2,8 +2,6 @@ DateTime::sub() -- fall type2 type2 --CREDITS-- Daniel Convissor ---XFAIL-- -Various bugs exist --FILE-- -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php
[PHP-CVS] svn: /php/php-src/ branches/PHP_5_4/Zend/tests/bug60611.phpt branches/PHP_5_4/Zend/zend_compile.c trunk/Zend/tests/bug60611.phpt trunk/Zend/zend_compile.c
laruence Wed, 28 Dec 2011 06:46:12 +
Revision: http://svn.php.net/viewvc?view=revision&revision=321458
Log:
Fix #60613 (Segmentation fault with $cls->{expr}() syntax)
#now the behavior is like 5.3, a COMPILER_ERROR will be triggered
Bug: https://bugs.php.net/60613 (Assigned) Segmentation fault with
$cls->{expr}() syntax
Changed paths:
U php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt
U php/php-src/branches/PHP_5_4/Zend/zend_compile.c
U php/php-src/trunk/Zend/tests/bug60611.phpt
U php/php-src/trunk/Zend/zend_compile.c
Modified: php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt
===
--- php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt 2011-12-28
04:31:21 UTC (rev 321457)
+++ php/php-src/branches/PHP_5_4/Zend/tests/bug60611.phpt 2011-12-28
06:46:12 UTC (rev 321458)
@@ -9,12 +9,6 @@
}
}
-Cls::{0}();
-Cls::{1.0}();
-Cls::{true}();
-Cls::{false}();
-Cls::{null}();
-
$cls = new Cls;
$cls->{0}();
$cls->{1.0}();
@@ -22,7 +16,12 @@
$cls->{false}();
$cls->{null}();
-echo "done";
+Cls::{0}();
+Cls::{1.0}();
+Cls::{true}();
+Cls::{false}();
+Cls::{null}();
+
?>
---EXPECT--
-done
+--EXPECTF--
+Fatal error: Method name must be a string in %sbug60611.php on line %d
Modified: php/php-src/branches/PHP_5_4/Zend/zend_compile.c
===
--- php/php-src/branches/PHP_5_4/Zend/zend_compile.c2011-12-28 04:31:21 UTC
(rev 321457)
+++ php/php-src/branches/PHP_5_4/Zend/zend_compile.c2011-12-28 06:46:12 UTC
(rev 321458)
@@ -1975,8 +1975,9 @@
zval name;
name = CONSTANT(last_op->op2.constant);
if (Z_TYPE(name) != IS_STRING) {
- convert_to_string(&name);
- } else if (!IS_INTERNED(Z_STRVAL(name))) {
+ zend_error(E_COMPILE_ERROR, "Method name must
be a string");
+ }
+ if (!IS_INTERNED(Z_STRVAL(name))) {
Z_STRVAL(name) = estrndup(Z_STRVAL(name),
Z_STRLEN(name));
}
FREE_POLYMORPHIC_CACHE_SLOT(last_op->op2.constant);
@@ -2369,8 +2370,8 @@
if (method_name->op_type == IS_CONST) {
char *lcname;
- if (Z_TYPE(method_name->u.constant) != IS_STRING) {
- convert_to_string(&method_name->u.constant);
+ if (Z_TYPE(method_name->u.constant) != IS_STRING) {
+ zend_error(E_COMPILE_ERROR, "Method name must be a
string");
}
lcname =
zend_str_tolower_dup(Z_STRVAL(method_name->u.constant),
Z_STRLEN(method_name->u.constant));
if ((sizeof(ZEND_CONSTRUCTOR_FUNC_NAME)-1) ==
Z_STRLEN(method_name->u.constant) &&
Modified: php/php-src/trunk/Zend/tests/bug60611.phpt
===
--- php/php-src/trunk/Zend/tests/bug60611.phpt 2011-12-28 04:31:21 UTC (rev
321457)
+++ php/php-src/trunk/Zend/tests/bug60611.phpt 2011-12-28 06:46:12 UTC (rev
321458)
@@ -9,12 +9,6 @@
}
}
-Cls::{0}();
-Cls::{1.0}();
-Cls::{true}();
-Cls::{false}();
-Cls::{null}();
-
$cls = new Cls;
$cls->{0}();
$cls->{1.0}();
@@ -22,7 +16,12 @@
$cls->{false}();
$cls->{null}();
-echo "done";
+Cls::{0}();
+Cls::{1.0}();
+Cls::{true}();
+Cls::{false}();
+Cls::{null}();
+
?>
---EXPECT--
-done
+--EXPECTF--
+Fatal error: Method name must be a string in %sbug60611.php on line %d
Modified: php/php-src/trunk/Zend/zend_compile.c
===
--- php/php-src/trunk/Zend/zend_compile.c 2011-12-28 04:31:21 UTC (rev
321457)
+++ php/php-src/trunk/Zend/zend_compile.c 2011-12-28 06:46:12 UTC (rev
321458)
@@ -1975,8 +1975,9 @@
zval name;
name = CONSTANT(last_op->op2.constant);
if (Z_TYPE(name) != IS_STRING) {
- convert_to_string(&name);
- } else if (!IS_INTERNED(Z_STRVAL(name))) {
+ zend_error(E_COMPILE_ERROR, "Method name must
be a string");
+ }
+ if (!IS_INTERNED(Z_STRVAL(name))) {
Z_STRVAL(name) = estrndup(Z_STRVAL(name),
Z_STRLEN(name));
}
FREE_POLYMORPHIC_CACHE_SLOT(last_op->op2.constant);
@@ -2369,8 +2370,8 @@
if (method_name->op_type == IS_CONST) {
char *lcname;
- if (Z_TYPE(method_name->u.constant) != IS_STRING) {
- convert_to_string(&method_name->u.constant);
+ if (Z_TYPE(method_name->u.constant) != IS_STRING) {
+ zend_error(E_COMPILE_ERR
