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 -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
