[PATCH] D56943: [clang-format][NFC] Allow getLLVMStyle() to take a language

[Jordan Rupprecht via Phabricator via cfe-commits]

rupprecht added inline comments.


================
Comment at: clang/lib/Format/ContinuationIndenter.cpp:193
                               RawStringFormat.Language, &PredefinedStyle)) {
-        PredefinedStyle = getLLVMStyle();
+        PredefinedStyle = getLLVMStyle(FormatStyle::LK_Cpp);
         PredefinedStyle.Language = RawStringFormat.Language;
----------------
MyDeveloperDay wrote:
> 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.
My goal was to fix all the in-tree callers but leave it as an optional arg (at least initially) so as to not break out of tree callers. I reduced it to what I actually need for this patch.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56943/new/

https://reviews.llvm.org/D56943