Fix #1743: Add pattern for struct constructor initializers.

ICSharpCode.Decompiler.Tests/TestCases/Pretty/ConstructorInitializers.cs

@@ -20,6 +20,22 @@ namespace ICSharpCode.Decompiler.Tests.TestCases.Pretty
 {
 	public class ConstructorInitializers
 	{
+		public struct Issue1743
+		{
+			public int Leet;
+
+			public Issue1743(int dummy)
+				: this(dummy, dummy)
+			{
+				Leet += dummy;
+			}
+
+			public Issue1743(int dummy1, int dummy2)
+			{
+				Leet = dummy1 + dummy2;
+			}
+		}
+
 		public struct SimpleStruct
 		{
 			public int Field1;

ICSharpCode.Decompiler/CSharp/Transforms/ConvertConstructorCallIntoInitializer.cs

@@ -58,30 +58,55 @@ namespace ICSharpCode.Decompiler.CSharp.Transforms
 		
 		public override void VisitConstructorDeclaration(ConstructorDeclaration constructorDeclaration)
 		{
-			var stmt = constructorDeclaration.Body.Statements.FirstOrDefault() as ExpressionStatement;
+			if (!(constructorDeclaration.Body.Statements.FirstOrDefault() is ExpressionStatement stmt))
-			if (stmt == null)
 				return;
-			if (!(stmt.Expression is InvocationExpression invocation))
+			var currentCtor = (IMethod)constructorDeclaration.GetSymbol();
-				return;
+			ConstructorInitializer ci;
-			if (invocation.Target is MemberReferenceExpression mre && mre.MemberName == ".ctor") {
+			switch (stmt.Expression) {
-				ConstructorInitializer ci = new ConstructorInitializer();
+				// Pattern for reference types:
-				var target = mre.Target;
+				// this..ctor(...);
-				// Ignore casts, those might be added if references are missing.
+				case InvocationExpression invocation:
-				if (target is CastExpression cast)
+					if (!(invocation.Target is MemberReferenceExpression mre) || mre.MemberName != ".ctor")
-					target = cast.Expression;
+						return;
-				if (target is ThisReferenceExpression)
+					if (!(invocation.GetSymbol() is IMethod ctor && ctor.IsConstructor))
-					ci.ConstructorInitializerType = ConstructorInitializerType.This;
+						return;
-				else if (target is BaseReferenceExpression)
+					ci = new ConstructorInitializer();
-					ci.ConstructorInitializerType = ConstructorInitializerType.Base;
+					var target = mre.Target;
-				else
+					// Ignore casts, those might be added if references are missing.
-					return;
+					if (target is CastExpression cast)
-				// Move arguments from invocation to initializer:
+						target = cast.Expression;
-				invocation.Arguments.MoveTo(ci.Arguments);
+					if (target is ThisReferenceExpression)
-				// Add the initializer: (unless it is the default 'base()')
+						ci.ConstructorInitializerType = ConstructorInitializerType.This;
-				if (!(ci.ConstructorInitializerType == ConstructorInitializerType.Base && ci.Arguments.Count == 0))
+					else if (target is BaseReferenceExpression)
-					constructorDeclaration.Initializer = ci.CopyAnnotationsFrom(invocation);
+						ci.ConstructorInitializerType = ConstructorInitializerType.Base;
-				// Remove the statement:
+					else
-				stmt.Remove();
+						return;
+					// Move arguments from invocation to initializer:
+					invocation.Arguments.MoveTo(ci.Arguments);
+					// Add the initializer: (unless it is the default 'base()')
+					if (!(ci.ConstructorInitializerType == ConstructorInitializerType.Base && ci.Arguments.Count == 0))
+						constructorDeclaration.Initializer = ci.CopyAnnotationsFrom(invocation);
+					// Remove the statement:
+					stmt.Remove();
+					break;
+				// Pattern for value types:
+				// this = new TSelf(...);
+				case AssignmentExpression assignment:
+					if (!(assignment.Right is ObjectCreateExpression oce && oce.GetSymbol() is IMethod ctor2 && ctor2.DeclaringTypeDefinition == currentCtor.DeclaringTypeDefinition))
+						return;
+					ci = new ConstructorInitializer();
+					if (assignment.Left is ThisReferenceExpression)
+						ci.ConstructorInitializerType = ConstructorInitializerType.This;
+					else
+						return;
+					// Move arguments from invocation to initializer:
+					oce.Arguments.MoveTo(ci.Arguments);
+					// Add the initializer: (unless it is the default 'base()')
+					if (!(ci.ConstructorInitializerType == ConstructorInitializerType.Base && ci.Arguments.Count == 0))
+						constructorDeclaration.Initializer = ci.CopyAnnotationsFrom(oce);
+					// Remove the statement:
+					stmt.Remove();
+					break;
 			}
 		}