sstwcw added a comment. In D147422#4240024 <https://reviews.llvm.org/D147422#4240024>, @MyDeveloperDay wrote:
> In D147422#4240021 <https://reviews.llvm.org/D147422#4240021>, @owenpan wrote: > >> Should we extend `SpacesInContainerLiterals` so that it controls JSON colons >> too? If yes, then we don't need `SpaceBeforeJsonColon`. Otherwise, IMO we >> should leave the doc alone. > > My concern for `SpacesInContainerLiterals` is that it impacts arrays contents > too '[ 1, 2, 3 ]'. It looks like a line break gets inserted in arrays. Does that mean the option doesn't affect arrays? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147422/new/ https://reviews.llvm.org/D147422 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
