emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Add notifications.el


From: Michael Albinus
Subject: Re: [PATCH] Add notifications.el
Date: Thu, 10 Jun 2010 10:26:56 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux)

Julien Danjou <address@hidden> writes:

> It seems to clear to me now that people may want to identify the
> notification by id to use with their close/actions function, so let's
> make that change until it's too late.

Your patch looks reasonable to me. If you have a ChangeLog entry, I'm
willing to commit it.

(And maybe you have also docstrings for `notifications-on-action-signal'
and `notifications-on-closed-signal')

Best regards, Michael.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]