Return-Path: Date: Tue, 15 Sep 2009 20:55:23 +0300 From: Johan Hedberg To: Marcel Holtmann Cc: =?iso-8859-1?Q?RISK=D3?= Gergely , Luiz Augusto von Dentz , linux-bluetooth@vger.kernel.org, Context Devel mailing list Subject: Re: [PATCH] Add introspection interface to the output of introspection calls. Message-ID: <20090915175523.GA8393@jh-x301> References: <87k50hoee1.fsf@bubble.risko.hu> <2d5a2c100909020746t63bcd89bj1b25d053a6f4f9ca@mail.gmail.com> <87pra9nspo.fsf@bubble.risko.hu> <20090914141816.GA17774@jh-x301> <871vm9fuis.fsf@bubble.risko.hu> <20090914211145.GA16947@jh-x301> <2d5a2c100909150350r2e453cb6h5c4fe95c8638f9e9@mail.gmail.com> <87ws40e6iz.fsf@bubble.risko.hu> <20090915152240.GA4378@jh-x301> <1253035625.28416.2.camel@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1253035625.28416.2.camel@localhost.localdomain> List-ID: Hi Marcel, On Tue, Sep 15, 2009, Marcel Holtmann wrote: > what are we gaining from doing it like this? I really don't see the > benefit of doing it. Someone please explain it to me. It does add more > code than it deletes. It's about avoiding special-casing the Introspect method and using the framework that gdbus already has for method callbacks. This feels like a cleaner approach to me and with the two refactoring tasks I suggested I suspect we'd also be reducing the total line count (the current patch adds 15 lines to the total but there's something like 15-20 new lines in it that unnecessarily duplicate exsiting code). Johan