Hi,

The attached patch fixes some problems I found in Gapi while generating
the bindings for Glade-3. Is it ok to commit?

Thanks,
Lluis.

Index: generator/FieldBase.cs
===================================================================
--- generator/FieldBase.cs	(revisión: 54370)
+++ generator/FieldBase.cs	(copia de trabajo)
@@ -195,7 +195,7 @@
 					sw.WriteLine (indent + "\t\t\t*raw_ptr = value;");
 				} else {
 					sw.WriteLine (indent + "\t\t\t" + table.GetMarshalReturnType (CType) + "* raw_ptr = (" + table.GetMarshalReturnType (CType) + "*)(((byte*)" + container_type.CallByName () + ") + " + offsetName + ");");
-					sw.WriteLine (indent + "\t\t\t*raw_ptr = " + table.CallByName (ctype, "value") + ";");
+					sw.WriteLine (indent + "\t\t\t*raw_ptr = " + table.ToNativeReturn (ctype, "value") + ";");
 				}
 				sw.WriteLine (indent + "\t\t}");
 				sw.WriteLine (indent + "\t}");
Index: generator/StructField.cs
===================================================================
--- generator/StructField.cs	(revisión: 54370)
+++ generator/StructField.cs	(copia de trabajo)
@@ -133,7 +133,7 @@
 				sw.WriteLine (indent + "private {0} {1};", CSType, Name);
 				sw.WriteLine ();
 				if (Access != "private") {
-					sw.WriteLine (indent + Access + " " + wrapped + " " + wrapped_name + " {");
+					sw.WriteLine (indent + Access + " " + wrapped + " " + StudlyName + " {");
 					sw.WriteLine (indent + "\tget { return " + table.FromNativeReturn (CType, Name) + "; }");
 					sw.WriteLine (indent + "}");
 				}
Index: generator/ManualGen.cs
===================================================================
--- generator/ManualGen.cs	(revisión: 54370)
+++ generator/ManualGen.cs	(copia de trabajo)
@@ -46,7 +46,7 @@
 
 		public override string CallByName (string var_name)
 		{
-			return var_name + " == null ? IntPtr.Zero : " + var_name + ".Handle";
+			return "(" + var_name + " == null ? IntPtr.Zero : " + var_name + ".Handle)";
 		}
 		
 		public override string FromNative(string var)
Index: ChangeLog
===================================================================
--- ChangeLog	(revisión: 54370)
+++ ChangeLog	(copia de trabajo)
@@ -1,3 +1,11 @@
+2005-12-14 Lluis Sanchez Gual <[EMAIL PROTECTED]>
+
+	* generator/FieldBase.cs: Properly convert marshalled value 
+	to native value.
+	* generator/StructField.cs: Don't use the C name as field name.
+	* generator/ManualGen.cs: Add brackets to the result of CallByName,
+	since it can cause problems in some contexts.
+
 2005-12-13  Mike Kestner  <[EMAIL PROTECTED]>
 
 	* sources/Makefile.am : move to stable gtk 2.8 versions.
_______________________________________________
Gtk-sharp-list maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/gtk-sharp-list

Reply via email to