PropertiesChanged signal, take 2

[David Zeuthen]

Hey,

On Tue, May 11, 2010 at 4:41 PM, Colin Walters <walters at verbum.org> wrote:
> On Tue, May 11, 2010 at 2:07 PM, David Zeuthen <zeuthen at gmail.com> wrote:
>> Hey again,
>>
>> Here's take two of the PropertiesChanged spec patch reworked after
>> review here on the list (thanks guys!). Notes:
>
> This looks fine to me in general.   You need to bump the
> <releaseinfo>, and while we don't have a formal mechanism for this
> it'd be good to add a note like "This signal was added in version
> 0.13".

(I think you meant 0.14 because that's what I bumped the version to.
It was at 0.13 already.)

OK, I committed the attached patch (which contains the change Thiago
wanted; one annotation instead of two). I included a lot of the
information in this thread in the commit summit.

What's the process for uploading the spec it so it's accessible when visiting

 http://dbus.freedesktop.org/doc/dbus-specification.html

which is 0.12? (this one is pretty ancient... it doesn't mention
nonce-tcp: for instance)

> Looking at the spec, I wonder if any existing binding actually looks
> at the NoReply annotation at runtime.  I highly doubt it...

Well, NoReply only means the _service_ won't reply. Callers always
need to check for a reply. For starters, the bus might cut you off....
if it runs out of memory or resources.

Thanks,
David
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-an-optional-PropertiesChanged-signal.patch
Type: application/octet-stream
Size: 3991 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dbus/attachments/20100512/23b73ea1/attachment.obj>