Bug 2023 – Returning from foreach body doesn't work as expected.

The bug is reproduced by the following code. Instead of making foo return, the 'return' statement in foo's foreach body breaks the loop: ---- struct S { int a[]; int opApply(int delegate(ref int) dg) { foreach (x; a) { if(dg(x)) return 1; } return 0; } } void foo() { auto s = S([1, 2, 3]); foreach (x; s) { if (x == 1) return; // breaks the loop instead of returning from foo. } assert(false); // should never reach here. } void main() { foo(); } ----

Here's my opportunity to restate my opinion that the programmer should not be responsible for passing around opaque magic values generated by the compiler. At least not in a supposedly modern language. I wrote a proposal about how to make opApply more user-friendly Here: digitalmars.D:64686 And a few clarifications here: digitalmars.D:64706 Unfortunately I think it really requires macro() to make it pretty.

(In reply to comment #1) I think there's some way to get links to NG posts here automatically but I can never remember what it is... > I wrote a proposal about how to make opApply more user-friendly > Here: digitalmars.D:64686 http://www.digitalmars.com/pnews/read.php?server=news.digitalmars.com&group=digitalmars.D&artnum=64686&header > And a few clarifications here: digitalmars.D:64706 http://www.digitalmars.com/pnews/read.php?server=news.digitalmars.com&group=digitalmars.D&artnum=64706&header

Maybe it makes sense to post your proposal to bugzilla as enhancement? For now, I'm closing the bug as the example is invalid.