rajatbajpai added inline comments.
================ Comment at: clang/unittests/Format/FormatTest.cpp:14545 verifyFormat("auto lambda = []() { return 0; };", SomeSpace2); + + FormatStyle SpaceAfterOperatorOverloading = getLLVMStyle(); ---------------- HazardyKnusperkeks wrote: > rajatbajpai wrote: > > MyDeveloperDay wrote: > > > There should be a PARSE unit test too please > > I'm sorry if I misunderstood it, do you mean tests using CHECK_PARSE? I am > > confirming this because I didn't find any such test case for existing > > //SpaceBeforeParensOptions// options. > Than this has slipped through, but there should be a test for every parsing > aspect. Thanks for confirming. Not an issue, I'll try to add a test case for existing options as well then. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116283/new/ https://reviews.llvm.org/D116283 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits