I need to check b) will work with dereferencing and isset - but yes, I think it will probably be the
best way to go for now.
the NOTICE is probably just a missing 'E_' so it should be E_NOTICE - I had a typo there
at one point.
Regards
Alan
--- On 07/Dec/2011, Daniel Convissor wrote:
> Hi Alan:
>
> > b) fix isset, and return first character
> > ISSET: return false (no warning)
> > READ: warning + return first character (similar to current behavior
> > except dereferenced strings)
> >
> > c) fix isset, and return empty strings
> > ISSET: return false (no warning)
> > READ: warning + return empty string
>
> While "c" is cleaner, "b" is less of a change, so it seems that's the
> best way to go. What do other people think?
>
> FYI, when I applied your existing patch to trunk, the following build
> error comes up:
>
> /php/php/php-src/trunk/Zend/zend_execute.c: In function
> âzend_fetch_dimension_addressâ:
> /php/php/php-src/trunk/Zend/zend_execute.c:1164: error: âNOTICEâ
> undeclared (first use in this function)
> /php/php/php-src/trunk/Zend/zend_execute.c:1164: error: (Each
> undeclared identifier is reported only once
> /php/php/php-src/trunk/Zend/zend_execute.c:1164: error: for each
> function it appears in.)
>
> Thanks,
>
> --Dan
>
> --
> T H E A N A L Y S I S A N D S O L U T I O N S C O M P A N Y
> data intensive web and database programming
> http://www.AnalysisAndSolutions.com/
> 4015 7th Ave #4, Brooklyn NY 11232 v: 718-854-0335 f: 718-854-0409
>
> --
> PHP Internals - PHP Runtime Development Mailing List
> To unsubscribe, visit: http://www.php.net/unsub.php