Bug->Req #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: dmitry@php.net Reported by: derick@php.net Summary: Compiler creates (unused) compiled variables for self::$foo Status: Assigned -Type: Bug +Type: Feature/Change Request 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: yeah. the fix requires significant modification of grammar and compiler. It would be good to fix it, but I'm not sure it costs the time. In general it's not a bug, but a feature request. Previous Comments: ------------------------------------------------------------------------ [2011-09-15 03:59:43] laruence@php.net 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. ------------------------------------------------------------------------ [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 ------------------------------------------------------------------------ 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