MyDeveloperDay added inline comments.
Comment at: clang/unittests/Format/FormatTest.cpp:14545
verifyFormat("auto lambda = []() { return 0; };", SomeSpace2);
+
+ FormatStyle SpaceAfterOperatorOverloading = getLLVMStyle();
There should be a PARSE unit test too
rajatbajpai updated this revision to Diff 396314.
rajatbajpai added a comment.
Fixed the patch file.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116283/new/
https://reviews.llvm.org/D116283
Files:
clang/docs/ClangFormatStyleOptions.rst
rajatbajpai updated this revision to Diff 396302.
rajatbajpai added a comment.
Incorporated review comments.
Changelog:
1. Using unit test case instead of lit test case.
2. Placed the option according to alphabetical order.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
HazardyKnusperkeks added inline comments.
Comment at: clang/include/clang/Format/Format.h:3383
+///true: false:
+///void operator++ (int a); vs.void operator++(int
+///a);
Please put
MyDeveloperDay added a comment.
you need to add a unit test int clang/unittest/Format/FormatTest.cpp (we tend
to not use lit tests)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116283/new/
https://reviews.llvm.org/D116283
rajatbajpai created this revision.
rajatbajpai added a reviewer: MyDeveloperDay.
rajatbajpai added a project: clang-format.
rajatbajpai requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This change adds an option AfterOperatorOverloading in