Tickets :: Comment on [#6837] Singletons and references

* Your Email Address
* Spam protection	Enter the letters below: .__..__..___._..__ \| \|\| \|[__ \| \| \ \|__\|\|__\|\| _\|_\|__/
Comment	>> PHP 5.2.x currently doesn't handle cleanup of cyclic references. PHP > >> 5.3.x should be fixed, but it doesn't work properly. I filed an > >> upstream bug: http://bugs.php.net/?id=45178 > > > > Your upstream bug contains an incorrect statement: > > The supplied code is the standard way Horde does it singletons. We > always used the syntax of "$object = &new class" to make it work > with PHP4 and PHP5. > > > > As Chuck mentioned, the code you provided in that bug report is most > definitely not the "standard" syntax we use. If that syntax does > appear anywhere, it is an error and needs to be fixed (thus this bug > report). Regardless of whether PHP is leaking memory or not, this > needs to be cleaned up regardless if just because semantically it is > incorrect.
Attachment
Watch this ticket