<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Mar 26, 2014 at 11:06 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 26 March 2014 13:36, Eli Bendersky <<a href="mailto:eliben@google.com">eliben@google.com</a>> wrote:<br>


> Hello,<br>
><br>
> This patch is in similar vein to what done earlier to<br>
> Module::globals/aliases etc. I would really like to iterate over function<br>
> arguments like this in new code:<br>
><br>
>   for (Argument Arg : F.args()) {<br>
>     ...<br>
>   }<br>
<br>
<br>
</div>That is fine.<br>
<div class=""><br>
> The question is: would it be OK to change existing code from explicit<br>
> iteration with the .args_begin/args_end idiom to this new idiom?<br>
<br>
</div>Probably yes, we have been changing many other iterators, this should<br>
probably not be an exception.<br><br></blockquote><div><br></div><div>Thanks for the review, all. Committed in r204835. I won't do large-scale changes just for this reason, but will feel free to convert locally in code touched by existing patches.</div>

<div><br></div><div>Eli</div><div> </div></div><br></div></div>