From: Luca Bruno <lucabru@src.gnome.org>
Date: Sat, 1 Feb 2014 17:37:50 +0000 (+0100)
Subject: Move PostfixExpression transformation into the transformer
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=16f415bd684673bc71cabb8a33e95bb706707c02;p=thirdparty%2Fvala.git

Move PostfixExpression transformation into the transformer
---

diff --git a/codegen/valaccodebasemodule.vala b/codegen/valaccodebasemodule.vala
index c68e9863d..ca2c4e91c 100644
--- a/codegen/valaccodebasemodule.vala
+++ b/codegen/valaccodebasemodule.vala
@@ -4403,47 +4403,6 @@ public abstract class Vala.CCodeBaseModule : CodeGenerator {
 		}
 	}
 
-	public override void visit_postfix_expression (PostfixExpression expr) {
-		MemberAccess ma = find_property_access (expr.inner);
-		if (ma != null) {
-			// property postfix expression
-			var prop = (Property) ma.symbol_reference;
-
-			// increment/decrement property
-			var op = expr.increment ? CCodeBinaryOperator.PLUS : CCodeBinaryOperator.MINUS;
-			var cexpr = new CCodeBinaryExpression (op, get_cvalue (expr.inner), new CCodeConstant ("1"));
-			store_property (prop, ma.inner, new GLibValue (expr.value_type, cexpr));
-
-			// return previous value
-			expr.target_value = expr.inner.target_value;
-			return;
-		}
-
-		// assign current value to temp variable
-		var temp_value = store_temp_value (expr.inner.target_value, expr);
-
-		// increment/decrement variable
-		var op = expr.increment ? CCodeBinaryOperator.PLUS : CCodeBinaryOperator.MINUS;
-		var cexpr = new CCodeBinaryExpression (op, get_cvalue_ (temp_value), new CCodeConstant ("1"));
-		ccode.add_assignment (get_cvalue (expr.inner), cexpr);
-
-		// return previous value
-		expr.target_value = temp_value;
-	}
-
-	private MemberAccess? find_property_access (Expression expr) {
-		if (!(expr is MemberAccess)) {
-			return null;
-		}
-
-		var ma = (MemberAccess) expr;
-		if (ma.symbol_reference is Property) {
-			return ma;
-		}
-
-		return null;
-	}
-
 	static bool is_limited_generic_type (GenericType type) {
 		unowned Class? cl = type.type_parameter.parent_symbol as Class;
 		unowned Struct? st = type.type_parameter.parent_symbol as Struct;
diff --git a/codegen/valaccodetransformer.vala b/codegen/valaccodetransformer.vala
index 3c55c7792..56578683a 100644
--- a/codegen/valaccodetransformer.vala
+++ b/codegen/valaccodetransformer.vala
@@ -367,4 +367,16 @@ public class Vala.CCodeTransformer : CodeTransformer {
 
 		end_replace_expression (replacement);
 	}
+
+	public override void visit_postfix_expression (PostfixExpression expr) {
+		begin_replace_expression (expr);
+
+		var result = b.add_temp_declaration (copy_type (expr.value_type), expr.inner);
+		var op = expr.increment ? "+ 1" : "- 1";
+		b.add_expression (expression (@"$(expr.inner) = $result $op"));
+
+		var replacement = return_temp_access (result, expr.value_type, expr.target_type);
+
+		end_replace_expression (replacement);
+	}
 }