2004-10-27 Joe Shaw <joeshaw@novell.com>

	* mono/Arguments.cs (GetDBusTypeConstructor):
	type.UnderlyingSystemType will return "System.Byte" if you do it
	on "byte[]", which is not what we want.  So check the type.IsArray
	property and use System.Array instead.

diff --git a/ChangeLog b/ChangeLog
index c8f2349..8260ce2 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2004-10-27  Joe Shaw  <joeshaw@novell.com>
+
+       * mono/Arguments.cs (GetDBusTypeConstructor):
+       type.UnderlyingSystemType will return "System.Byte" if you do it
+       on "byte[]", which is not what we want.  So check the type.IsArray
+       property and use System.Array instead.
+
 2004-10-25  John (J5) Palmieri  <johnp@redhat.com>
 
        * dbus/dbus-sysdeps.c (fill_user_info): On errors do not free

diff --git a/mono/Arguments.cs b/mono/Arguments.cs
index ca178ae..d78fbff 100644 (file)
--- a/mono/Arguments.cs
+++ b/mono/Arguments.cs
@@ -161,7 +161,14 @@ namespace DBus
     // Get the appropriate constructor for a D-BUS type
     public static ConstructorInfo GetDBusTypeConstructor(Type dbusType, Type type) 
     {
-      ConstructorInfo constructor = dbusType.GetConstructor(new Type[] {type.UnderlyingSystemType, typeof(Service)});
+      Type constructorType;
+
+      if (type.IsArray)
+        constructorType = typeof (System.Array);
+      else
+        constructorType = type.UnderlyingSystemType;
+
+      ConstructorInfo constructor = dbusType.GetConstructor(new Type[] {constructorType, typeof(Service)});
       if (constructor == null)
        throw new ArgumentException("There is no valid constructor for '" + dbusType + "' from type '" + type + "'");