================ @@ -10093,11 +10090,11 @@ TEST_F(FormatTest, ReturnTypeBreakingStyle) { getGoogleStyleWithColumns(40)); verifyFormat("Tttttttttttttttttttttttt ppppppppppppppp\n" " ABSL_GUARDED_BY(mutex1)\n" - " ABSL_GUARDED_BY(mutex2);", + " ABSL_GUARDED_BY(mutex2);", ---------------- ilya-biryukov wrote:
It is wrong, but it was like the recent patch where I changed Google style. I mentioned it in the commit description (quote follows): > Also update the test with two GUARDED_BY directives. While the formatting > after > https://github.com/llvm/llvm-project/commit/efeb546865c233dfa7706ee0316c676de9f69897 > seems better, this case is rare enough to not warrant the extra complexity. > We are reverting it back to the state it had before https://github.com/llvm/llvm-project/commit/efeb546865c233dfa7706ee0316c676de9f69897. https://github.com/llvm/llvm-project/pull/76804 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits