1) The new syntax makes mess for me.
2) After (1) I don't see any reason in limitation to jump only down,
Both of these items were based on the PDM meeting notes which required this
specific behavior.
2) Your patch doesn't care about freeing loop variables
and we got a memory leaks in the following example
I was thinking zend_brk_cont() took care of all the containers, forgot about
the outtermost block.
3) You can declare one LABEL twice
Doi.. Good point.
4) Keeping "labels" field in op_array after compilation make no sense
Agreed, but CG(labels) wouldn't necessarily work as you could be parsing a
function while in the middle of parsing the main scope and they'd clobber
over each other. Freeing it at the end of pass_two() is a step in the right
direction of course.
But I still not sure that PHP needs "goto".
TBH, I was surprised to see this on the table in the first place myself...
-Sara
--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php