dgoldman added inline comments.

================
Comment at: clang/lib/Sema/SemaCodeComplete.cpp:5199
+      // Objective-C property reference. Bail if we're performing fix-it code
+      // completion since we don't handle forwarding the fix-it for Objective-C
+      // objects. Since Objective-C properties are normally backed by ivars,
----------------
sammccall wrote:
> I don't really follow what the comment means by "forwarding the fixit". As I 
> understood it, the issue is -> and . are likely both valid, so changing one 
> to the other is confusing. Might be able to reword to clarify.
I meant the code previously would try give fix it code completions without 
marking it as a fix it. But I've gone ahead and removed it in favor of the 
other comment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81263



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

Reply via email to