This revision was automatically updated to reflect the committed changes.
Closed by commit rL307812: [clang-tidy] Ignore blank spaces between cast's ")"
and its sub expr. (authored by hokein).
Repository:
rL LLVM
https://reviews.llvm.org/D31700
Files:
clang-tools-extra/trunk/clang-tidy/goog
alexfh accepted this revision.
alexfh added a comment.
LG. Thanks!
https://reviews.llvm.org/D31700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
hokein added a comment.
> You might want to commit this at some point ;]
I somewhat forgot this minor patch.
Comment at: clang-tidy/google/AvoidCStyleCastsCheck.cpp:139
")");
+ ReplaceRange = CharSourceRange::getCharRange(CastExpr->getLParenLoc(),
+
hokein updated this revision to Diff 106219.
hokein marked an inline comment as done.
hokein added a comment.
Herald added subscribers: xazax.hun, JDevlieghere.
Support more cases.
https://reviews.llvm.org/D31700
Files:
clang-tidy/google/AvoidCStyleCastsCheck.cpp
test/clang-tidy/google-read
alexfh added a comment.
You might want to commit this at some point ;]
https://reviews.llvm.org/D31700
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
alexfh accepted this revision.
alexfh added a comment.
This revision is now accepted and ready to land.
Sorry, missed your patch somehow. LG with one nit.
Comment at: clang-tidy/google/AvoidCStyleCastsCheck.cpp:139
")");
+ ReplaceRange = CharSourceRange::getChar