[vala] Fix disconnecting from detailed signal

[Jürg Billeter <juergbi src gnome org>]

commit 1b93e3119c731f92ca517e17853180dd8e80c59f
Author: Jonathan Conder <j skurvy no-ip org>
Date:   Sat Mar 20 13:53:03 2010 +0100

    Fix disconnecting from detailed signal
    
    Fixes bug 602293.

 codegen/valagsignalmodule.vala |   26 +++++++++++++++++++-------
 1 files changed, 19 insertions(+), 7 deletions(-)
---
diff --git a/codegen/valagsignalmodule.vala b/codegen/valagsignalmodule.vala
index 593f0ce..8dda3ec 100644
--- a/codegen/valagsignalmodule.vala
+++ b/codegen/valagsignalmodule.vala
@@ -611,6 +611,8 @@ internal class Vala.GSignalModule : GObjectModule {
 			ccall.add_argument (get_result_cexpression ("self"));
 		}
 
+		CCodeCommaExpression? ccomma = null;
+
 		if (sig is DynamicSignal) {
 			// dynamic_signal_connect or dynamic_signal_disconnect
 
@@ -632,7 +634,7 @@ internal class Vala.GSignalModule : GObjectModule {
 			}
 
 			// get signal id
-			var ccomma = new CCodeCommaExpression ();
+			ccomma = new CCodeCommaExpression ();
 			var temp_decl = get_temp_variable (uint_type);
 			temp_vars.insert (0, temp_decl);
 			var parse_call = new CCodeFunctionCall (new CCodeIdentifier ("g_signal_parse_name"));
@@ -640,22 +642,27 @@ internal class Vala.GSignalModule : GObjectModule {
 			var decl_type = (TypeSymbol) sig.parent_symbol;
 			parse_call.add_argument (new CCodeIdentifier (decl_type.get_type_id ()));
 			parse_call.add_argument (new CCodeUnaryExpression (CCodeUnaryOperator.ADDRESS_OF, get_variable_cexpression (temp_decl.name)));
+			LocalVariable? detail_temp_decl = null;
 			if (signal_detail == null) {
 				parse_call.add_argument (new CCodeConstant ("NULL"));
+				parse_call.add_argument (new CCodeConstant ("FALSE"));
 			} else {
-				var detail_temp_decl = get_temp_variable (gquark_type);
+				detail_temp_decl = get_temp_variable (gquark_type);
 				temp_vars.insert (0, detail_temp_decl);
 				parse_call.add_argument (new CCodeUnaryExpression (CCodeUnaryOperator.ADDRESS_OF, new CCodeIdentifier (detail_temp_decl.name)));
+				parse_call.add_argument (new CCodeConstant ("TRUE"));
 			}
-			parse_call.add_argument (new CCodeConstant ("FALSE"));
 			ccomma.append_expression (parse_call);
-			ccomma.append_expression (get_variable_cexpression (temp_decl.name));
 
 			// third argument: signal_id
-			ccall.add_argument (ccomma);
+			ccall.add_argument (get_variable_cexpression (temp_decl.name));
 
 			// fourth argument: detail
-			ccall.add_argument (new CCodeConstant ("0"));
+			if (detail_temp_decl == null) {
+				ccall.add_argument (new CCodeConstant ("0"));
+			} else {
+				ccall.add_argument (get_variable_cexpression (detail_temp_decl.name));
+			}
 			// fifth argument: closure
 			ccall.add_argument (new CCodeConstant ("NULL"));
 		}
@@ -711,7 +718,12 @@ internal class Vala.GSignalModule : GObjectModule {
 			ccall.add_argument (new CCodeConstant ("NULL"));
 		}
 
-		return ccall;
+		if (ccomma != null) {
+			ccomma.append_expression (ccall);
+			return ccomma;
+		} else {
+			return ccall;
+		}
 	}
 }