Re: cvs: php-src /ext/mysqlnd mysqlnd_wireprotocol.c

From: Date: Thu, 11 Jun 2009 17:47:22 +0000
Subject: Re: cvs: php-src /ext/mysqlnd mysqlnd_wireprotocol.c
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-58651@lists.php.net to get a copy of this message
Hi Pierre, Pierre-Alain Joye wrote:
pajoye Thu Jun 11 17:16:35 2009 UTC
  Modified files:                  /php-src/ext/mysqlnd	mysqlnd_wireprotocol.c   Log:
- nuke unused var http://cvs.php.net/viewvc.cgi/php-src/ext/mysqlnd/mysqlnd_wireprotocol.c?r1=1.29&r2=1.30&diff_format=u Index: php-src/ext/mysqlnd/mysqlnd_wireprotocol.c diff -u php-src/ext/mysqlnd/mysqlnd_wireprotocol.c:1.29 php-src/ext/mysqlnd/mysqlnd_wireprotocol.c:1.30 --- php-src/ext/mysqlnd/mysqlnd_wireprotocol.c:1.29 Thu Jun 11 17:14:34 2009 +++ php-src/ext/mysqlnd/mysqlnd_wireprotocol.c Thu Jun 11 17:16:35 2009 @@ -1364,7 +1364,6 @@ zval **current_field, **end_field, **start_field; zend_bool as_unicode = conn->options.numeric_and_datetime_as_unicode; zend_bool allocated; - void *obj; DBG_ENTER("php_mysqlnd_rowp_read_binary_protocol");
I don't like the change and will revert it. In any case, next time you do a change to mysqlnd please merge it with the other branch. In this case change was in HEAD and not merged into PHP5_3. This is not the first time I see this happening. mysqlnd in 5_3 and HEAD _should_ be in sync - exactly the same code except for the cvs tags. This makes the life easier. Andrey

« previous php.cvs (#58651) next »