brianchandotcom/liferay-portal — pull request #8325 by shuyangzhou (2013-01-04 01:46:09+00:00)
Title: LPS-32059 Fix broken loop control flow

Ok, this is a special case. Because there is no post processing logic in the bottom of the outter loop, so we can directly do a return inside the inner loop. This can eliminate both the "break label" and control variable.

Just to confirm, are we supposed to never use "break label"? Because there will be more complex cases, where break label will be the only clean solution. The "break label" is basically a simplified version of goto for Java, much safer than goto, and of course much less powerful. Should we always prefer control variable over break label?