Re: Re: com php-src: Remove support for classes without class entries: Zend/zend_API.c Zend/zend_builtin_functions.c Zend/zend_closures.c
Zend/zend_execute.h Zend/zend_interfaces.c Zend/zend_object_handlers.c Zend/zend_object_handlers.h Zend/zend...

From: Date: Thu, 09 Oct 2014 22:59:41 +0000
Subject: Re: Re: com php-src: Remove support for classes without class entries: Zend/zend_API.c Zend/zend_builtin_functions.c Zend/zend_closures.c
Zend/zend_execute.h Zend/zend_interfaces.c Zend/zend_object_handlers.c Zend/zend_object_handlers.h Zend/zend...
References: 1 2 3 4 5  Groups: php.cvs php.internals 
Request: Send a blank email to php-cvs+get-82229@lists.php.net to get a copy of this message
Hi On 10 Oct 2014, at 03:57, Dmitry Stogov <dmitry@zend.com> wrote: > Yeah, JavaBridge used it, but it doesn't means it need it. > > zend_class_entry * _php_java_get_class_entry(zval *object TSRMLS_DC) > { > return java_class; > } > > I agree, it would be better if we would discuss it on @internals, but it > would take much longer time > If you still see use cases for it, please show If anything, could this be recorded in the UPGRADING file? Thanks > > Thanks. Dmitry. > > > On Thu, Oct 9, 2014 at 11:48 PM, Stas Malyshev <smalyshev@sugarcrm.com> > wrote: > >> Hi! >> >>> we discussed this issue with Nikita on IRC, and didn't find any use >>> cases for objects without ce now. >>> Nor in php neither in pecl. >>> Do you see any problems? >> >> Those were used to create "bridge" APIs - like Zend's Java bridge (not >> sure if supported now, probably not, and if it's still uses that), as >> far as I remember. It may be that PECL does not use it but people have >> internal extensions that are not published. So deleting a chunk of the >> API without as much as a note on internals is not really a good idea, I >> think. >> >> IRC discussion is not a public discussion, since most of the people are >> not present on the IRC and have no idea about anything being discussed >> there. >> -- >> Stanislav Malyshev, Software Architect >> SugarCRM: http://www.sugarcrm.com/ >>

« previous php.cvs (#82229) next »