[PATCH] D69164: [clang-format] fix regression recognizing casts in Obj-C calls

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 18 11:35:00 PDT 2019


krasimir marked an inline comment as done.
krasimir added inline comments.


================
Comment at: lib/Format/TokenAnnotator.cpp:1612
+                          Keywords.kw_final) ||
+        isCpp11AttributeSpecifier(*Tok.Next))
       return false;
----------------
rdwampler wrote:
> I think the issue r373922 was fixing is only related to the `delete`. Can this check be move further up where we explicitly check if the token is the delete keyword? 
> 
Sorry, I didn't notice this comment until now. I can investigate whether all the other cases for non-delete methods are already covered, it would be interesting. + @MyDeveloperDay, who might have better insights into this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69164





More information about the cfe-commits mailing list