From f5bb7fc1b08cd2600135f0cf98ff53b23d1cc78c Mon Sep 17 00:00:00 2001 From: Colin Walters Date: Thu, 16 Jun 2005 19:45:49 +0000 Subject: 2005-06-16 Colin Walters * glib/dbus-gobject.c (funcsig_hash, funcsig_equal): Use n_params to iterate instead of walking to G_TYPE_INVALID. Patch based on a patch from Ryan Gammon. --- glib/dbus-gobject.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) (limited to 'glib/dbus-gobject.c') diff --git a/glib/dbus-gobject.c b/glib/dbus-gobject.c index da5da3b2..8e43155a 100644 --- a/glib/dbus-gobject.c +++ b/glib/dbus-gobject.c @@ -1314,11 +1314,12 @@ funcsig_hash (gconstpointer key) const DBusGFuncSignature *sig = key; GType *types; guint ret; + guint i; ret = sig->rettype; types = sig->params; - while (*types != G_TYPE_INVALID) + for (i = 0; i < sig->n_params; i++) { ret += (int) (*types); types++; @@ -1335,22 +1336,22 @@ funcsig_equal (gconstpointer aval, const DBusGFuncSignature *b = bval; const GType *atypes; const GType *btypes; + guint i, j; - if (a->rettype != b->rettype) + if (a->rettype != b->rettype + || a->n_params != b->n_params) return FALSE; atypes = a->params; btypes = b->params; - while (*atypes != G_TYPE_INVALID) + for (i = 0; i < a->n_params; i++) { if (*btypes != *atypes) return FALSE; atypes++; btypes++; } - if (*btypes != G_TYPE_INVALID) - return FALSE; return TRUE; } -- cgit