[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Add notifications.el
From: |
Julien Danjou |
Subject: |
Re: [PATCH] Add notifications.el |
Date: |
Mon, 07 Jun 2010 17:59:29 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) |
On Mon, Jun 07 2010, Michael Albinus wrote:
> They are not mandatory functionally (there is no D-Bus error), but as
> you say, the result is weird. I like Stefan's idea to make them mandatory
> parameters; if somebody really wants to left them empty, she could pass "".
If you really want, I will amend the patch to add at list `:title' as
mandatory. OTOH body is really optionnal, quoting specs[1]:
``If the body is omitted, just the summary is displayed.''
> Grrr. I need new glasses!
The fact you missed it also proves it might be so obvious at first
read. ;)
> Maybe you have disabled D-Bus error in one way or the other, I've got an
> error. Did you run dbus-monitor in parallel, during tests?
Yes, and I really saw nothing. I don't think I disabled any D-Bus
error. I just saw an empty array [].
[1] http://www.galago-project.org/specs/notification/0.9/x81.html
Anyhow, thanks much for the review Michael.
Cheers,
--
Julien Danjou
// ᐰ <address@hidden> http://julien.danjou.info
pgpOCGkJQgXaU.pgp
Description: PGP signature
- [PATCH] Add notifications.el, Julien Danjou, 2010/06/05
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/07
- Re: [PATCH] Add notifications.el, Stefan Monnier, 2010/06/07
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/07
- [PATCH] Add notifications.el, Julien Danjou, 2010/06/07
- Re: [PATCH] Add notifications.el, Davis Herring, 2010/06/07
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/08
- [PATCH] Add notifications.el, Julien Danjou, 2010/06/08
- Re: [PATCH] Add notifications.el, Michael Albinus, 2010/06/08
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/08
- Re: [PATCH] Add notifications.el, Jan Moringen, 2010/06/10
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/11
- Message not available
- Re: [PATCH] Add notifications.el, Jan Moringen, 2010/06/11
- Re: [PATCH] Add notifications.el, Julien Danjou, 2010/06/11