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

This revision was automatically updated to reflect the committed changes. Closed by commit rG4977fd2192fc: [clang-format] Missing space between trailing return type auto and left brace (authored by denis-fatkulin, committed by ArcsinX). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

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

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

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

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); +

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

HazardyKnusperkeks added a comment. You should mark comments as done, if so. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130417/new/ https://reviews.llvm.org/D130417 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

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

denis-fatkulin updated this revision to Diff 447116. denis-fatkulin added a comment. Test cases for `auto &` and `auto *` are added CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130417/new/ https://reviews.llvm.org/D130417 Files: clang/lib/Format/TokenAnnotator.cpp

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

curdeius accepted this revision. curdeius added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:23555-23556 + FormatStyle Style = getLLVMStyle(); + verifyFormat("[]() -> auto { return Val; }", Style); + verifyFormat("auto foo() -> auto { return Val; }",

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

owenpan accepted this revision. owenpan added inline comments. Comment at: clang/unittests/Format/FormatTest.cpp:23555-23556 + FormatStyle Style = getLLVMStyle(); + verifyFormat("[]() -> auto { return Val; }", Style); + verifyFormat("auto foo() -> auto { return Val; }",

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

HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130417/new/ https://reviews.llvm.org/D130417

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

denis-fatkulin created this revision. denis-fatkulin added reviewers: HazardyKnusperkeks, curdeius. denis-fatkulin added a project: clang-format. Herald added a project: All. denis-fatkulin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits.