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

Reply via email to