Fix duplicate ResolveResult on ConvertConstantValue()

ICSharpCode.Decompiler/CSharp/Annotations.cs

@@ -57,6 +57,13 @@ namespace ICSharpCode.Decompiler.CSharp
 			return new TranslatedExpression(expression.Expression, expression.ResolveResult);
 		}
 		
+		public static TranslatedExpression WithILInstruction(this ExpressionWithResolveResult expression, IEnumerable<ILInstruction> instructions)
+		{
+			foreach (var inst in instructions)
+				expression.Expression.AddAnnotation(inst);
+			return new TranslatedExpression(expression.Expression, expression.ResolveResult);
+		}
+		
 		public static TranslatedExpression WithoutILInstruction(this ExpressionWithResolveResult expression)
 		{
 			return new TranslatedExpression(expression.Expression, expression.ResolveResult);

ICSharpCode.Decompiler/CSharp/ExpressionBuilder.cs

@@ -40,7 +40,7 @@ namespace ICSharpCode.Decompiler.CSharp
 	{
 		internal readonly ICompilation compilation;
 		internal readonly CSharpResolver resolver;
-		internal readonly TypeSystemAstBuilder astBuilder;
+		readonly TypeSystemAstBuilder astBuilder;
 		
 		public ExpressionBuilder(ITypeResolveContext decompilationContext)
 		{
@@ -55,10 +55,21 @@ namespace ICSharpCode.Decompiler.CSharp
 		public AstType ConvertType(IType type)
 		{
 			var astType = astBuilder.ConvertType(type);
-			astType.AddAnnotation(new TypeResolveResult(type));
+			Debug.Assert(astType.Annotation<TypeResolveResult>() != null);
 			return astType;
 		}
 		
+		public ExpressionWithResolveResult ConvertConstantValue(ResolveResult rr)
+		{
+			var expr = astBuilder.ConvertConstantValue(rr);
+			var exprRR = expr.Annotation<ResolveResult>();
+			if (exprRR == null) {
+				exprRR = rr;
+				expr.AddAnnotation(rr);
+			}
+			return new ExpressionWithResolveResult(expr, exprRR);
+		}
+		
 		public TranslatedExpression Translate(ILInstruction inst)
 		{
 			Debug.Assert(inst != null);

ICSharpCode.Decompiler/CSharp/TranslatedExpression.cs

@@ -172,9 +172,8 @@ namespace ICSharpCode.Decompiler.CSharp
 			}
 			var rr = expressionBuilder.resolver.ResolveCast(targetType, ResolveResult);
 			if (rr.IsCompileTimeConstant && !rr.IsError) {
-				return expressionBuilder.astBuilder.ConvertConstantValue(rr)
+				return expressionBuilder.ConvertConstantValue(rr)
-					.WithILInstruction(this.ILInstructions)
+					.WithILInstruction(this.ILInstructions);
-					.WithRR(rr);
 			}
 			return Expression.CastTo(expressionBuilder.ConvertType(targetType))
 				.WithoutILInstruction().WithRR(rr);

ILSpy/Properties/app.config.template

@@ -13,7 +13,7 @@
 			</dependentAssembly>
 			<dependentAssembly>
 				<assemblyIdentity name="ICSharpCode.AvalonEdit" publicKeyToken="9cc39be672370310" culture="neutral"/>
-				<bindingRedirect oldVersion="4.1.0.0-99.9.9.9" newVersion="4.4.2.9744"/>
+				<bindingRedirect oldVersion="4.1.0.0-99.9.9.9" newVersion="5.0.1.0"/>
 			</dependentAssembly>
 			<dependentAssembly>
 				<assemblyIdentity name="ICSharpCode.NRefactory" publicKeyToken="d4bfe873e7598c49" culture="neutral"/>