<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Aug 19, 2009, at 2:06 PM, Fariborz Jahanian wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">I knew the general use. However, there is no break/continue in this manufactured for-loop. <div>Thus the comment. I don't think we need it, unless there is some implied assumption requiring them.</div></div></blockquote><div><br class="webkit-block-placeholder"></div>My apologies. My response was not relevant. I read too fast.</div><div><br class="webkit-block-placeholder"></div><div>- Fariborz</div><div><br><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><br class="webkit-block-placeholder"></div><div>- Thanks, Fariborz</div><div><br><div><div>On Aug 19, 2009, at 2:02 PM, Anders Carlsson wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Aug 19, 2009, at 1:55 PM, Fariborz Jahanian wrote:</div><blockquote type="cite"><div><font class="Apple-style-span" color="#000000"><br></font>+ // Store the blocks to use for break and continue.<br>+ // FIXME. Is this needed?<br>+ BreakContinueStack.push_back(BreakContinue(AfterFor, ContinueBlock));<br></div></blockquote><div><br></div>It's not needed. BreakContinueStack is used so we know where to jump when generating break and continue statements.</div><div><blockquote type="cite"><div><font class="Apple-style-span" color="#000000"><br></font></div><div>+ // FIXME. Do we need this?<br>+ BreakContinueStack.pop_back();<br></div></blockquote><div><br></div>Same here.</div><div><br></div><div>Anders</div></div></blockquote></div><br></div></div>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>