arphaman added a comment.

In https://reviews.llvm.org/D39730#926108, @jkorous-apple wrote:

> Sorry, what do you mean by "this works in the body of the class"?
>
> Definition of constructor inside class definition has been working even 
> before:
>
>   struct foo {
>     foo();
>     f<CODE_COMPLETE_HERE>
>   };
>   
>
> Do you mean that a test shall be added in order to check that it still works?


You're right, but we don't actually code-complete constructor in this case, but 
the struct name. I suppose we don't need to complete the constructor here, but 
we do need the destructor, i.e.:

  struct foo {
  ~<CODE_COMPLETE_HERE>
  };


https://reviews.llvm.org/D39730



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to