[PATCH] D133589: [clang-format] JSON formatting add new option for controlling newlines in json arrays

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 12 00:13:17 PDT 2022


MyDeveloperDay marked 6 inline comments as done.
MyDeveloperDay added inline comments.


================
Comment at: clang/include/clang/Format/Format.h:2076
+  /// \version 16
+  bool BreakArrays;
+
----------------
HazardyKnusperkeks wrote:
> HazardyKnusperkeks wrote:
> > Please sort.
> What about `Leave`?
I'm going to say that at present the boolean is probably enough as JSON formatting is really to do one or the other, but of course this can progress easily into an enum when there is a use case


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

https://reviews.llvm.org/D133589



More information about the cfe-commits mailing list