Gio::DBus::InterfaceVTable: Make the tuple clearer.

* gio/src/dbusinterfacevtable.[hg|ccg]: SlotInterfaceMethodCall: Change
the VariantBase type to a VariantContainerBase, because this is a tuple,
so people should get the children. This makes it more obvious.
DBusInterfaceVTable_MethodCall_giomm_callback(): Adapt.
Suggested by Michael Edwards.

3 files changed, 14 insertions, 2 deletions

diff --git a/ChangeLog b/ChangeLog
index 661ca44b..36363e0d 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,15 @@
 2011-03-23  Murray Cumming  <murrayc@murrayc.com>
 
+	Gio::DBus::InterfaceVTable: Make the tuple clearer.
+
+	* gio/src/dbusinterfacevtable.[hg|ccg]: SlotInterfaceMethodCall: Change
+	the VariantBase type to a VariantContainerBase, because this is a tuple,
+	so people should get the children. This makes it more obvious.
+	DBusInterfaceVTable_MethodCall_giomm_callback(): Adapt.
+	Suggested by Michael Edwards.
+
+2011-03-23  Murray Cumming  <murrayc@murrayc.com>
+
 	Gio::DBus: Move InterfaceVTable and SubtreeVTable to their own files.
 
 	* gio/src/dbusinterfacevtable.[hg|ccg]:
diff --git a/gio/src/dbusinterfacevtable.ccg b/gio/src/dbusinterfacevtable.ccg
index bf478708..4db46a2d 100644
--- a/gio/src/dbusinterfacevtable.ccg
+++ b/gio/src/dbusinterfacevtable.ccg
@@ -44,9 +44,11 @@ static void DBusInterfaceVTable_MethodCall_giomm_callback(
 
   try
   {
+    // Note that we use VariantContainerBase instead of VariantBase for the
+    // GVariant, because it is documented as being a tuple (list of values).
     (*the_slot)(Glib::wrap(connection, true), (sender ? sender : ""),
       object_path, interface_name, method_name,
-      Glib::VariantBase(parameters, true), Glib::wrap(invocation, true));
+      Glib::VariantContainerBase(parameters, true), Glib::wrap(invocation, true));
   }
   catch(...)
   {
diff --git a/gio/src/dbusinterfacevtable.hg b/gio/src/dbusinterfacevtable.hg
index 5a49b018..a572af7c 100644
--- a/gio/src/dbusinterfacevtable.hg
+++ b/gio/src/dbusinterfacevtable.hg
@@ -69,7 +69,7 @@ public:
     const Glib::ustring&,
     const Glib::ustring&,
     const Glib::ustring&,
-    const Glib::VariantBase&,
+    const Glib::VariantContainerBase&,
     const Glib::RefPtr<MethodInvocation>&
   > SlotInterfaceMethodCall;