[PATCH] D130417: [clang-format] Missing space between trailing return type 'auto' and left brace

[Denis Fatkulin via Phabricator via cfe-commits]

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