On Sun, Mar 18, 2007 at 08:48:31PM -0400, Wez Furlong wrote:

> Updated patch at http://pastebin.ca/400952

This is interesting.  I'm not sure I'll ever use it (I prefer named
functions), but it's a purely optional and folks seem to want
something like this.

One comment on the patch itself:

    static unsigned int anon_count;

I think you should provide an initial value (0?) for this static
counter.  Also, you should protect against wrap-around (which, while
improbably, is possible).

Alternatively, you could change the anonymous function naming scheme
to something like __zend_anon_FILE_LINE_COLUMN, but that could be an
unnecessary waste of string memory.

-- 
Jon Parise (jon of php.net) :: The PHP Project (http://www.php.net/)

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to