[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