[PATCH] D125171: [clang-format] Add a new clang-format option AlwaysBreakBeforeFunctionParameters

[jonathan molinatto via Phabricator via cfe-commits]

jrmolin added inline comments.


================
Comment at: clang/include/clang/Format/Format.h:851
+  /// The function parameter breaking style to use.
+  /// \version 16.0
+  FunctionParameterBreakingStyle BreakBeforeFunctionParameters;
----------------
so should this be 17.0 now? This has gone on quite long.


================
Comment at: clang/lib/Format/Format.cpp:345
+
+    // For backward compatibility.
+    IO.enumCase(Value, "false", FormatStyle::FPBS_Leave);
----------------
I should probably delete these, right?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125171