[prev in list] [next in list] [prev in thread] [next in thread] 

List:       pecl-cvs
Subject:    Re: [PECL-CVS] cvs: pecl /memcache memcache.c php_memcache.h  /memcache/tests
From:       Antony Dovgal <antony () zend ! com>
Date:       2005-11-28 22:14:04
Message-ID: 438B8EA0.1000806 () zend ! com
[Download RAW message or body]

On 29.11.2005 01:03, Mikael Johansson wrote:
> @@ -320,10 +320,6 @@
> static void mmc_server_free(mmc_t *mmc TSRMLS_DC) /* {{{ */
> {
> 	if (mmc->persistent) {
> -		if (mmc->stream != NULL) {
> -			/* persistent streams are closed by the engine */
> -			/* php_stream_pclose(mmc->stream); */
> -		}

It's fine to remove it, but I wanted to leave it just for clarification, because it \
really looks non-obvious.

-- 
Wbr, 
Antony Dovgal

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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic