Re: defadvice file-expand-wildcards ad-do-it value

[Michael Albinus]

Kevin Ryde <address@hidden> writes:

> In the unload, I think after ad-remove-advice that
>
>     (ad-activate 'file-expand-wildcards)
>
> is also needed to actually update the function's definition (per my
> second diff).

Indeed. Committed.

(Somehow, I failed to read your second diff. Sorry for that)

> (I've never understood why those add/remove/enable/disable funcs don't
> update the func immediately.  I suppose it saves some cycles not to
> update in between multiple updates, but the scope for not getting what
> you mean is a bit high!)

Same here. I always need several attempts with advice functions, until
they do what i want.

Thanks, and best regards, Michael.