Re: [libvirt] [dbus PATCH 14/18] connect: parse message arguments as first thing

2018-03-13 Thread Daniel P . Berrangé
On Mon, Mar 12, 2018 at 05:21:44PM +0100, Pavel Hrdina wrote: > There is no need to open connection if parsing arguments fails. > > Signed-off-by: Pavel Hrdina > --- > src/connect.c | 12 ++-- > 1 file changed, 6 insertions(+), 6 deletions(-) Reviewed-by: Daniel P. Berrangé Regards,

[libvirt] [dbus PATCH 14/18] connect: parse message arguments as first thing

2018-03-12 Thread Pavel Hrdina
There is no need to open connection if parsing arguments fails. Signed-off-by: Pavel Hrdina --- src/connect.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/connect.c b/src/connect.c index b31f08e..695fc0d 100644 --- a/src/connect.c +++ b/src/connect.c @@ -1