From ccff28f4ff8eb4b9f730ff8497b957077b51e28f Mon Sep 17 00:00:00 2001 From: Bela VanderVoort Date: Mon, 9 Jan 2023 09:31:18 -0600 Subject: [PATCH] Put constructor initializer on new line (#782) closes #526 --- .../TestFiles/ConstructorDeclarations.cst | 18 ++++++++++++------ .../BaseMethodDeclaration.cs | 2 +- 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/Src/CSharpier.Tests/FormattingTests/TestFiles/ConstructorDeclarations.cst b/Src/CSharpier.Tests/FormattingTests/TestFiles/ConstructorDeclarations.cst index d46d6c6a7..b1b5e91c4 100644 --- a/Src/CSharpier.Tests/FormattingTests/TestFiles/ConstructorDeclarations.cst +++ b/Src/CSharpier.Tests/FormattingTests/TestFiles/ConstructorDeclarations.cst @@ -23,9 +23,11 @@ public class BasicClass public class Initializers : BasicClass { - public Initializers() : this(true) { } + public Initializers() + : this(true) { } - public Initializers(string value) : base(value) { } + public Initializers(string value) + : base(value) { } public Initializers( string longParameter_______________________________________________________________________________ @@ -46,11 +48,13 @@ public class Initializers : BasicClass public Initializers( string longParameter____________________________________________________________ - ) : base(false) { } + ) + : base(false) { } public Initializers( string longParameter____________________________________________________________ - ) : base(false) + ) + : base(false) { WithBody(); } @@ -94,9 +98,11 @@ public class Initializers : BasicClass public LongParametersWithEmptyBody(string parameter, bool anotherParameter__________________) { } - public LongInitializerWithEmptyBody() : base(parameter, anotherParameter___________________) { } + public LongInitializerWithEmptyBody() + : base(parameter, anotherParameter___________________) { } - public LongInitializerWithEmptyBody() : base(parameter, anotherParameter__________________) { } + public LongInitializerWithEmptyBody() + : base(parameter, anotherParameter__________________) { } public LongInitializerWithEmptyBody() : base(parameter______________, anotherParameter___________________________________________) diff --git a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseMethodDeclaration.cs b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseMethodDeclaration.cs index 307ffed91..742cf98b3 100644 --- a/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseMethodDeclaration.cs +++ b/Src/CSharpier/SyntaxPrinter/SyntaxNodePrinters/BaseMethodDeclaration.cs @@ -180,7 +180,7 @@ operatorDeclarationSyntax.ExplicitInterfaceSpecifier is not null declarationGroup.Add( Doc.Group( - Doc.Indent(Doc.Line), + Doc.Indent(Doc.HardLine), Doc.Indent(colonToken), Token.Print(constructorInitializer.ThisOrBaseKeyword, context), Doc.Indent(argumentList)