Re: [PATCH BlueZ 1/7] gdbus: Add g_dbus_client_get_proxy
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: 2013-01-13 16:09:47
Hi Marcel, On Fri, Jan 11, 2013 at 10:46 PM, Marcel Holtmann [off-list ref] wrote:
Hi Luiz,quoted
g_dbus_client_get_proxy gives the possibilitity to just check if a proxy exist for the given path and interface pair instead of using g_dbus_proxy_new which end up creating a proxy if it doesn't exists which is not always necessary.why would we do that. You get the proxy via the client callbacks for proxy created or proxy removed.
That way we the user don't have to maintain a list of found proxies, such a list exist and is maintained by GDBusClient.
The proxy_new method is for dealing with services that do not have ObjectManager support.
Yep, so in one way or the other you are already letting the user application search in the list of proxies maintained by GDBusClient, the only thing this does is to make the proxy_lookup public so the client don't have to maintain its how list only to be able to look back when a proxy point to another, btw the reason proxy_new is not enough is because it may lead to extra calls when the user application may just want to bail out if the proxy is not found. -- Luiz Augusto von Dentz