kevinl created this revision.
kevinl added a reviewer: djasper.
Herald added subscribers: cfe-commits, klimek.
C++ code that used to be formatted as `if (! + object) {` is now formatted as
`if (!+object) {`
(we have a particular object in our codebase where unary `operator+` is
overloaded to ret
kevinl updated this revision to Diff 134511.
kevinl edited the summary of this revision.
kevinl added a comment.
Simplified the heuristic logic, which extends the fix to TypeScript as well
Repository:
rC Clang
https://reviews.llvm.org/D43312
Files:
lib/Format/TokenAnnotator.cpp
unittests
kevinl added inline comments.
Comment at: lib/Format/TokenAnnotator.cpp:1497
+!(PrevToken->is(tok::exclaim) &&
+ Style.Language == FormatStyle::LK_JavaScript))
// There aren't any trailing unary operators except for TypeScript's
krasimir w
kevinl added a comment.
I do not have commit access.
Would appreciate you committing it for me. Thanks!
Repository:
rC Clang
https://reviews.llvm.org/D43312
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/ma
kevinl added a comment.
@krasimir ping?
would appreciate you committing this for me. thanks!
Repository:
rC Clang
https://reviews.llvm.org/D43312
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi