Bug #55673 [Asn]: Compiler creates (unused) compiled variables for self::$foo

Edit report at https://bugs.php.net/bug.php?id=55673&edit=1 ID: 55673 Updated by: laruence@php.net Reported by: derick@php.net Summary: Compiler creates (unused) compiled variables for self::$foo Status: Assigned Type: Bug Package: Scripting Engine problem Operating System: * PHP Version: 5.3SVN-2011-09-12 (SVN) Assigned To: dmitry Block user comment: N Private report: N New Comment: this cv is created in fetch_simple_variable_ex, and fetch_simple_variable_ex is used by a lot of logic, so I think we should alter the parse.y to make the class::$static_member not to denpend on a reference_variable. I have tried, but bring in new reduce conflict. Previous Comments: ------------------------------------------------------------------------ [2011-09-14 15:29:34] laruence@php.net I'd better to report another one about the NOP opline, sorry agian. ------------------------------------------------------------------------ [2011-09-14 15:24:23] laruence@php.net OOPS!, I must lost my mind, what I was doing is erease NOP opline(god know How can I read this bug as "REMOVE UNUSED NOP" opline).... sorry, but maybe this patch is also a litte useful... ------------------------------------------------------------------------ [2011-09-14 15:10:14] laruence@php.net I have made a patch for this, and make whole test after patched. made sure that there is no new test failed after patched. TEST RESULT: trunk: http://pastebin.com/gMYc2Fp5 5.4branch: http://pastebin.com/7EePEE43 5.3branch: http://pastebin.com/m4wirXjr ------------------------------------------------------------------------ [2011-09-14 15:07:01] laruence@php.net The following patch has been added/updated: Patch Name: bug55673.diff Revision: 1316012821 URL: https://bugs.php.net/patch-display.php?bug=55673&patch=bug55673.diff&revision=1316012821 ------------------------------------------------------------------------ [2011-09-14 14:37:02] laruence@php.net after a quick digging, I found this is caused by zend_do_early_binding, which make the declare_class to NOP after use it. I will try to make a patch for this. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=55673 -- Edit this bug report at https://bugs.php.net/bug.php?id=55673&edit=1