MyDeveloperDay added inline comments. Herald added a subscriber: jdoerfert.
================ Comment at: clang/lib/Format/ContinuationIndenter.cpp:193 RawStringFormat.Language, &PredefinedStyle)) { - PredefinedStyle = getLLVMStyle(); + PredefinedStyle = getLLVMStyle(FormatStyle::LK_Cpp); PredefinedStyle.Language = RawStringFormat.Language; ---------------- if you've set up a default argument of LK_Cpp do you need to keep supplying it? if your going to supply it everywhere don't make it a default argument or vice versa This might make a much smaller patch, a little more digestable and all the tests can remain as they are. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D56943/new/ https://reviews.llvm.org/D56943 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits