Re: com php-src: Fix spl_directory cast_object, reenable error: Zend/zend_execute.h ext/spl/spl_directory.c

From: Date: Mon, 12 May 2014 19:37:45 +0000
Subject: Re: com php-src: Fix spl_directory cast_object, reenable error: Zend/zend_execute.h ext/spl/spl_directory.c
References: 1  Groups: php.cvs 
Request: Send a blank email to php-cvs+get-78291@lists.php.net to get a copy of this message
Hi Nikita, why do we need this new error message? Thanks. Dmitry. On Mon, May 12, 2014 at 10:44 PM, Nikita Popov <nikic@php.net> wrote: > Commit: 8904f72d7c47253345f7039afd8ca754442c7e34 > Author: Nikita Popov <nikic@php.net> Mon, 12 May 2014 20:44:30 > +0200 > Parents: f29b68306d457ebc157254a7af186e5a4fd6db80 > Branches: phpng > > Link: > > http://git.php.net/?p=php-src.git;a=commitdiff;h=8904f72d7c47253345f7039afd8ca754442c7e34 > > Log: > Fix spl_directory cast_object, reenable error > > Changed paths: > M Zend/zend_execute.h > M ext/spl/spl_directory.c > > > Diff: > diff --git a/Zend/zend_execute.h b/Zend/zend_execute.h > index 0007e98..52689d2 100644 > --- a/Zend/zend_execute.h > +++ b/Zend/zend_execute.h > @@ -115,8 +115,7 @@ again: > break; > } > > - // TODO: The following line > breakes phar ??? > - // zend_error(E_RECOVERABLE_ERROR, > "Object of class %s could not be converted to boolean", > Z_OBJ_P(op)->ce->name->val); > + zend_error(E_RECOVERABLE_ERROR, > "Object of class %s could not be converted to boolean", > Z_OBJ_P(op)->ce->name->val); > } else if (Z_OBJ_HT_P(op)->get) { > zval rv; > zval *tmp = > Z_OBJ_HT_P(op)->get(op, &rv TSRMLS_CC); > diff --git a/ext/spl/spl_directory.c b/ext/spl/spl_directory.c > index 8cc7ec0..f7a1de4 100644 > --- a/ext/spl/spl_directory.c > +++ b/ext/spl/spl_directory.c > @@ -1881,6 +1881,9 @@ static int spl_filesystem_object_cast(zval *readobj, > zval *writeobj, int type TS > } > return SUCCESS; > } > + } else if (type == _IS_BOOL) { > + ZVAL_TRUE(writeobj); > + return SUCCESS; > } > if (readobj == writeobj) { > zval_dtor(readobj); > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php > >

« previous php.cvs (#78291) next »