all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Why does advice-member-p accept only a SYMBOL
@ 2016-07-13 15:47 Michael Heerdegen
  2016-07-13 15:54 ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Heerdegen @ 2016-07-13 15:47 UTC (permalink / raw
  To: emacs-devel

Hi,

Why is advice-member-p limited to search only SYMBOL definitions
(i.e. symbol-functions)?  Wouldn't it be useful to have a predicate that
would accept a PLACE as in `add-function'?  AFAIK there is no such thing
in nadvice.el.


Thanks,

Michael.




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Why does advice-member-p accept only a SYMBOL
  2016-07-13 15:47 Why does advice-member-p accept only a SYMBOL Michael Heerdegen
@ 2016-07-13 15:54 ` Stefan Monnier
  2016-07-13 16:14   ` Michael Heerdegen
  0 siblings, 1 reply; 5+ messages in thread
From: Stefan Monnier @ 2016-07-13 15:54 UTC (permalink / raw
  To: emacs-devel

> Why is advice-member-p limited to search only SYMBOL definitions
> (i.e. symbol-functions)?  Wouldn't it be useful to have a predicate that
> would accept a PLACE as in `add-function'?  AFAIK there is no such thing
> in nadvice.el.

advice-function-member-p ?


        Stefan




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Why does advice-member-p accept only a SYMBOL
  2016-07-13 15:54 ` Stefan Monnier
@ 2016-07-13 16:14   ` Michael Heerdegen
  2016-07-13 16:43     ` Stefan Monnier
  2016-07-13 16:45     ` Stefan Monnier
  0 siblings, 2 replies; 5+ messages in thread
From: Michael Heerdegen @ 2016-07-13 16:14 UTC (permalink / raw
  To: emacs-devel

Stefan Monnier <monnier@iro.umontreal.ca> writes:

> advice-function-member-p ?

But that doesn't accept something like (var VARIABLE) or (local
VARIABLE) etc. as in `add-function', right?




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Why does advice-member-p accept only a SYMBOL
  2016-07-13 16:14   ` Michael Heerdegen
@ 2016-07-13 16:43     ` Stefan Monnier
  2016-07-13 16:45     ` Stefan Monnier
  1 sibling, 0 replies; 5+ messages in thread
From: Stefan Monnier @ 2016-07-13 16:43 UTC (permalink / raw
  To: emacs-devel

>> advice-function-member-p ?
> But that doesn't accept something like (var VARIABLE) or (local
> VARIABLE) etc. as in `add-function', right?

Not sure.  But if so, it's a bug.


        Stefan




^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Why does advice-member-p accept only a SYMBOL
  2016-07-13 16:14   ` Michael Heerdegen
  2016-07-13 16:43     ` Stefan Monnier
@ 2016-07-13 16:45     ` Stefan Monnier
  1 sibling, 0 replies; 5+ messages in thread
From: Stefan Monnier @ 2016-07-13 16:45 UTC (permalink / raw
  To: emacs-devel

>> advice-function-member-p ?
> But that doesn't accept something like (var VARIABLE) or (local
> VARIABLE) etc. as in `add-function', right?

Oh, yes, sorry, I double checked and you're right.  It works for most
places but not for the special ad-hoc places of add-function (aka `var'
and `local').


        Stefan




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-07-13 16:45 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-13 15:47 Why does advice-member-p accept only a SYMBOL Michael Heerdegen
2016-07-13 15:54 ` Stefan Monnier
2016-07-13 16:14   ` Michael Heerdegen
2016-07-13 16:43     ` Stefan Monnier
2016-07-13 16:45     ` Stefan Monnier

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.