[PATCH][review request] Use ArrayRef in SemaInit.cpp. Try 3

Dmitri Gribenko gribozavr at gmail.com
Fri May 3 08:07:10 PDT 2013


On Fri, May 3, 2013 at 5:09 PM, Robert Wilhelm <robert.wilhelm at gmx.net> wrote:
> Thanks for the review.
> Mutability is only needed in the ctor.
> I have attached new updated patch.

-Wdocumentation complains:

include/clang/Sema/Initialization.h:783:14: warning:
      parameter 'NumArgs' not found in the function declaration
[-Wdocumentation]
  /// \param NumArgs the number of arguments provided for initialization.
             ^~~~~~~

It is also possible to push ArrayRef'ization a bit to the beginning of
some functions to simplify code.

Committed r181022 with those cleanups.

Dmitri

--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the cfe-commits mailing list