[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Proposed new core library: alert.el
From: |
Artur Malabarba |
Subject: |
Re: Proposed new core library: alert.el |
Date: |
Fri, 6 Nov 2015 10:42:29 +0000 |
On 6 Nov 2015 9:47 am, "Rasmus" <address@hidden> wrote:
>
> John Wiegley <address@hidden> writes:
>
> >>>>>> raman <address@hidden> writes:
> >
> >> You could do this seamlessly by attaching an around advice to message and
> >> having that call alert
> >
> > Advice is only to be used by user code, not in core. It's the "last line" of
> > customization, but shouldn't be a programming style for developers.
>
> A rgrep on the lisp dir suggests that add-function is used 60 places.
> AFAIK, one of the intentions of nadvice is to easily attach new functions
> to "places", also in core. But I could easily be wrong.
add-function is about modifying function variables (i.e., places that are actually supposed to be configurable). It is not about advices.
I believe you're thinking of add-advice.
- Re: Posting new feature proposals on the wiki?, (continued)
- Re: Posting new feature proposals on the wiki?, Richard Stallman, 2015/11/12
- Re: Posting new feature proposals on the wiki?, John Wiegley, 2015/11/16
- Re: Posting new feature proposals on the wiki?, raman, 2015/11/16
- Re: Proposed new core library: alert.el, John Wiegley, 2015/11/06
- Re: Proposed new core library: alert.el, raman, 2015/11/05
- Re: Proposed new core library: alert.el, John Wiegley, 2015/11/05
- Re: Proposed new core library: alert.el, Rasmus, 2015/11/06
- Re: Proposed new core library: alert.el,
Artur Malabarba <=
- Re: Proposed new core library: alert.el, Xue Fuqiao, 2015/11/06
- Re: Proposed new core library: alert.el, Juanma Barranquero, 2015/11/06
- Re: Proposed new core library: alert.el, Eli Zaretskii, 2015/11/06
- Re: Proposed new core library: alert.el, Stefan Monnier, 2015/11/06
- Re: Proposed new core library: alert.el, raman, 2015/11/06
- Re: Proposed new core library: alert.el, John Wiegley, 2015/11/06
- Re: Proposed new core library: alert.el, T.V Raman, 2015/11/06