[PATCH] D130417: [clang-format] Missing space between trailing return type 'auto' and left brace
Denis Fatkulin via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 28 11:33:04 PDT 2022
denis-fatkulin updated this revision to Diff 448396.
denis-fatkulin added a comment.
Conflict with 'main' branch was solved
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130417/new/
https://reviews.llvm.org/D130417
Files:
clang/lib/Format/TokenAnnotator.cpp
clang/unittests/Format/FormatTest.cpp
Index: clang/unittests/Format/FormatTest.cpp
===================================================================
--- clang/unittests/Format/FormatTest.cpp
+++ clang/unittests/Format/FormatTest.cpp
@@ -23550,6 +23550,16 @@
verifyFormat("auto lambda = [&a = a]() { a = 2; };", AlignStyle);
}
+TEST_F(FormatTest, TrailingReturnTypeAuto) {
+ FormatStyle Style = getLLVMStyle();
+ verifyFormat("[]() -> auto { return Val; }", Style);
+ verifyFormat("[]() -> auto * { return Val; }", Style);
+ verifyFormat("[]() -> auto & { return Val; }", Style);
+ verifyFormat("auto foo() -> auto { return Val; }", Style);
+ verifyFormat("auto foo() -> auto * { return Val; }", Style);
+ verifyFormat("auto foo() -> auto & { return Val; }", Style);
+}
+
TEST_F(FormatTest, SpacesInConditionalStatement) {
FormatStyle Spaces = getLLVMStyle();
Spaces.IfMacros.clear();
Index: clang/lib/Format/TokenAnnotator.cpp
===================================================================
--- clang/lib/Format/TokenAnnotator.cpp
+++ clang/lib/Format/TokenAnnotator.cpp
@@ -3310,6 +3310,11 @@
}
}
+ // trailing return type 'auto': []() -> auto {}, auto foo() -> auto {}
+ if (Left.is(tok::kw_auto) &&
+ Right.isOneOf(TT_LambdaLBrace, TT_FunctionLBrace))
+ return true;
+
// auto{x} auto(x)
if (Left.is(tok::kw_auto) && Right.isOneOf(tok::l_paren, tok::l_brace))
return false;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D130417.448396.patch
Type: text/x-patch
Size: 1396 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220728/56eb9adc/attachment.bin>
More information about the cfe-commits
mailing list