denis-fatkulin marked 2 inline comments as done.
denis-fatkulin added a comment.
They marked done. Thanks!
================
Comment at: clang/unittests/Format/FormatTest.cpp:23555-23556
+ FormatStyle Style = getLLVMStyle();
+ verifyFormat("[]() -> auto { return Val; }", Style);
+ verifyFormat("auto foo() -> auto { return Val; }", Style);
+}
----------------
curdeius wrote:
> owenpan wrote:
> > Should we add test cases with an `&` between `auto` and `{`?
> :+1:
Then it's better to add the case not only for `auto &`, but for `auto *` too.
But actually these cases are hadled correctly without this patch.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130417/new/
https://reviews.llvm.org/D130417
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
