[PATCH] D79000: [clang-format] C# property formatting can be controlled by config options

Krasimir Georgiev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 28 09:06:42 PDT 2020


krasimir added a comment.

I'm curious why we don't go the opposite direction -- why are even those two styles needed separately?

  public int Style1 { get; set }
  // vs.
  public int Style2
  { get; set }

I'm sure there is a good reason; part of this is to make sure we document it so we may consistently do more updates and support more complicated cases in the same spirit.
IMO starting with some reasonable formatting, and later supporting different styles as needed, has been practical to keep clang-format from becoming harder to maintain.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79000





More information about the cfe-commits mailing list