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

Reply via email to