krasimir accepted this revision. krasimir added a comment. This revision is now accepted and ready to land.
I think this is OK as-is. @MyDeveloperDay's suggestion of adding AfterProperty to the BraceWrapping style makes sense. However I think in this case it is natural to use the same style for these properties and for methods (functions) until we have a reason to allow for divergence. This MS example <https://docs.microsoft.com/en-us/dotnet/csharp/programming-guide/classes-and-structs/properties#properties-with-backing-fields> illustrates how close such "property blocks" match "function body blocks": if you squint, add an `()` before the property block and sprinkle semicolons inside it, it looks like a method. The fact that we only need this specifically for C# properties makes it a bit niche. I'd rather be lazy about BraceWrapping.AfterProperty and add it later, as need arises. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79000/new/ https://reviews.llvm.org/D79000 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
