#46654 [NEW]: circular reference suggestion

From: luka8088 at gmail dot com Operating system: PHP version: 5.3.0alpha2 PHP Bug Type: Feature/Change Request Bug description: circular reference suggestion Description: ------------ I have a suggestion for circular reference problem, already mentioned in report 33595 ... I realize that it is already fix in cvs with David Wang's patch, but maybe this would also be good to have ? Reproduce code: --------------- for example: class foo { function __construct () { $this->bar = new bar($this); } function __refcount ($count) { if ($count <= 1) unset($this->bar); } } class bar { function __construct ($foo = null) { $this->foo = $foo; } } $object = new foo(); unset($object); // $object->__refcount(1); // one reference left Expected result: ---------------- now, this would create a circular reference when creating foo, and I suggest that when a reference to an object is removed or added, and refcount changes, a __refcount method is called, that way, in some cases when object knows how many circular references it has, can unset them and free the memory... in this example, foo knows that it has exactly 1 reference from bar, and if count is greater then 1, it is still needed... I know that with this, David Wang's patch may still be needed for objects that don't know the exact refcount, but this may have some good effect on performance ? rather then calculating circular references ? -- Edit bug report at http://bugs.php.net/?id=46654&edit=1 -- Try a CVS snapshot (PHP 5.2): http://bugs.php.net/fix.php?id=46654&r=trysnapshot52 Try a CVS snapshot (PHP 5.3): http://bugs.php.net/fix.php?id=46654&r=trysnapshot53 Try a CVS snapshot (PHP 6.0): http://bugs.php.net/fix.php?id=46654&r=trysnapshot60 Fixed in CVS: http://bugs.php.net/fix.php?id=46654&r=fixedcvs Fixed in CVS and need be documented: http://bugs.php.net/fix.php?id=46654&r=needdocs Fixed in release: http://bugs.php.net/fix.php?id=46654&r=alreadyfixed Need backtrace: http://bugs.php.net/fix.php?id=46654&r=needtrace Need Reproduce Script: http://bugs.php.net/fix.php?id=46654&r=needscript Try newer version: http://bugs.php.net/fix.php?id=46654&r=oldversion Not developer issue: http://bugs.php.net/fix.php?id=46654&r=support Expected behavior: http://bugs.php.net/fix.php?id=46654&r=notwrong Not enough info: http://bugs.php.net/fix.php?id=46654&r=notenoughinfo Submitted twice: http://bugs.php.net/fix.php?id=46654&r=submittedtwice register_globals: http://bugs.php.net/fix.php?id=46654&r=globals PHP 4 support discontinued: http://bugs.php.net/fix.php?id=46654&r=php4 Daylight Savings: http://bugs.php.net/fix.php?id=46654&r=dst IIS Stability: http://bugs.php.net/fix.php?id=46654&r=isapi Install GNU Sed: http://bugs.php.net/fix.php?id=46654&r=gnused Floating point limitations: http://bugs.php.net/fix.php?id=46654&r=float No Zend Extensions: http://bugs.php.net/fix.php?id=46654&r=nozend MySQL Configuration Error: http://bugs.php.net/fix.php?id=46654&r=mysqlcfg