================
@@ -3853,46 +3853,51 @@ struct FormatStyle {
     /// Leave definition blocks as they are.
     SDS_Leave,
     /// Insert an empty line between definition blocks.
-    SDS_Always,
+    SDS_One,
+    /// Insert two empty lines between definition blocks.
+    SDS_Two,
     /// Remove any empty line between definition blocks.
     SDS_Never
   };
 
   /// Specifies the use of empty lines to separate definition blocks, including
-  /// classes, structs, enums, and functions.
+  /// license text, includes, classes, structs, enums, and functions.
----------------
mydeveloperday wrote:
I am concerned by this approach, the others items there are often lost of items 
but for license block its likely one.

This feels like it should be handled seperately, with a new Options (coded 
seperately)

NewLinesAfterLicense: 2
NewLinesAfterIncludes: 1

I'd think about the Includes completely seperately as we might want to consider 
NewLines between IncludeGroups too.

But for me I think shoehorning this feature over the top of the SDS setting is 
unclear, and frankly a little smelly.

https://github.com/llvm/llvm-project/pull/77918
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to