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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 5 03:11:59 PDT 2020


MyDeveloperDay added a comment.

In D79000#2009813 <https://reviews.llvm.org/D79000#2009813>, @jbcoe wrote:

>   public int Style2
>   { get; set }
>
>
> appears in MS examples https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/properties
>
>   public int Style2 { get; set }
>
>
> is the style we use in our code (where the formatter will be put to immediate use).
>
> Other options can be added/supported (maybe a CSharpPropertyStyle enum?) but I don't think I'm going to have time to devote to that in the immediate future. Happy to review/advise as needed.


Maybe consider adding

AfterProperty to the BraceWrapping style?


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