Merge pull request #58 from mogmpoeira/master

garuma · web-flow · commit 0b83b901447b · 2017-10-07T13:20:41.000-04:00
Fixing Array Writing
diff --git a/src/Protocol/MessageWriter.cs b/src/Protocol/MessageWriter.cs
@@ -356,8 +356,13 @@ public void Write (object val)
 			}
 
 			Type type = val.GetType ();
-
-			WriteVariant (type, val);
+			
+			// TODO: workaround issue with array being written as variant
+			// See PR for context: https://github.com/mono/dbus-sharp/pull/58
+			if (type.IsArray)
+				Write (type, val);
+			else
+				WriteVariant (type, val);
 		}
 
 		public void WriteVariant (Type type, object val)

Original file line number	Diff line number	Diff line change
`@@ -356,8 +356,13 @@ public void Write (object val)`
`356`	`356`	`}`
`357`	`357`
`358`	`358`	`Type type = val.GetType ();`
`359`		`-`
`360`		`- WriteVariant (type, val);`
	`359`	`+`
	`360`	`+ // TODO: workaround issue with array being written as variant`
	`361`	`+ // See PR for context: https://github.com/mono/dbus-sharp/pull/58`
	`362`	`+ if (type.IsArray)`
	`363`	`+ Write (type, val);`
	`364`	`+ else`
	`365`	`+ WriteVariant (type, val);`
`361`	`366`	`}`
`362`	`367`
`363`	`368`	`public void WriteVariant (Type type, object val)`