<p dir="ltr">(Also, as an aside: please avoid changing the subject line in your emails (such as 'try 3" above) as it breaks threading in some email clients, like gmail)</p>
<div class="gmail_quote">On May 3, 2013 7:09 AM, "Robert Wilhelm" <<a href="mailto:robert.wilhelm@gmx.net">robert.wilhelm@gmx.net</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks for the review.<br>
Mutability is only needed in the ctor.<br>
I have attached new updated patch.<br>
<br>
Robert<br>
<br>
On Fri, 2013-05-03 at 16:08 +0300, Dmitri Gribenko wrote:<br>
> On Fri, May 3, 2013 at 3:59 PM, Robert Wilhelm <<a href="mailto:robert.wilhelm@gmx.net">robert.wilhelm@gmx.net</a>> wrote:<br>
> > Hello David,<br>
> ><br>
> >  This patch switches to ArrayRef in SemaInit.cpp.<br>
><br>
> Are you sure that we want to use MultiExprArg?  It is a<br>
> MutableArrayRef instead of a plain ArrayRef:<br>
><br>
> typedef llvm::MutableArrayRef<Expr*> MultiExprArg;<br>
><br>
> Is mutability needed here?<br>
><br>
> +  for (unsigned I = 0, J= Args.size(); I != J; ++I)<br>
><br>
> 'J' is not a good name for element count.  Please rename J to E and<br>
> add a space before '='.<br>
><br>
>        Expr *InitListAsExpr = InitList;<br>
> -      Expr **Args = InitList ? &InitListAsExpr : 0;<br>
> -      unsigned NumArgs = InitList ? 1 : 0;<br>
> +      MultiExprArg Args(&InitListAsExpr, InitList ? 1 : 0);<br>
><br>
> No tabs, please.<br>
><br>
> Dmitri<br>
><br>
> --<br>
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if<br>
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <<a href="mailto:gribozavr@gmail.com">gribozavr@gmail.com</a>>*/<br>
<br>
</blockquote></div>