dgoldman added a comment.

In D56924#1379342 <https://reviews.llvm.org/D56924#1379342>, @arphaman wrote:

> Do you know if this have an effect on the output of completion results or 
> other tooling-based output?
>
> A couple of requests:
>
> - This behavior should be controlled by a printing policy flag 
> `SupressUnwrittenScope` (specifically for the '::(class extension)').
> - I also agree with Sam's comment. The property should still be qualified by 
> the class name as well, e.g. `Obj::(class extension)::property`.


It's possible - clangd for instance was crashing because of the current 
behavior: ::property instead of Obj::property or (class extension)::property.

- Added although see below
- I've modified this to be more in line with the current handling of Objc 
methods, but this no longer outputs the "(class extension)", do you think it 
should output both?


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56924/new/

https://reviews.llvm.org/D56924



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

Reply via email to