[PATCH] Implementation for completion in call context for C++

Francisco Lopes da Silva francisco.mailing.lists at oblita.com
Tue Jan 13 14:14:36 PST 2015


In http://reviews.llvm.org/D6880#108168, @francisco.lopes wrote:

> Thanks for the feedback Manuel,
>
> I didn't try it in MSVC 2012, I hope it's not a hassle to compile clang in
>  Windows,
>  I'm downloading the express edition so I can try it.
>
> I believe the sole critical piece regarding the style guide is the
>  declaration of the
>  ParseExpressionList function. Although I think it's good to have the
>  default argument
>  initialized that way, which is simple and short, it may not be in
>  accordance with the
>  "Do not use Braced Initializer Lists to Call a Constructor" section of the
>  style guide.
>
> As I'm unsure about this specific usage, I'll left it as is until proper
>  review.
>
> 2015-01-13 17:12 GMT-02:00 Manuel Klimek <klimek at google.com>:
>
> > General high level comment: you're using C++11 extensively, have you made
>
> >  sure you're only using features MSVC 2012 supports (as per the style guide:
>
> >  http://llvm.org/docs/CodingStandards.html)?
>
> > 
>
> > http://reviews.llvm.org/D6880
>
> > 
>
> > EMAIL PREFERENCES
>
> > 
>
> >   http://reviews.llvm.org/settings/panel/emailpreferences/
>


Yeah, MSVC 2012 doesn't like copy-list-initialization either. Fixed, thanks again for the feedback.


http://reviews.llvm.org/D6880

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list