Re: Delayed warnings

[Chong Yidong]

Juanma Barranquero <address@hidden> writes:

> OK. I want to install this patch, with is a simplifed version of the
> previous one: `delayed-warnings-filter' does not exist, and
> `delayed-warnings-function' has become `delayed-warnings-hook', a true
> hook.
>
> The idea is that functions in the hook can process and remove as many
> elements from `delayed-warnings-list' as they want. The function
> `display-delayed-warnings', which is set by default to run last, of
> course removes and displays them all.
>
> That makes the delayed warnings infrastructure both simpler and more
> flexible. Redisplay warnings would mean (other than pushing the
> relevant warnings to `delayed-warnings-list', of course) just adding a
> new `display-redisplay-warnings' to the hook.

I think it's cleaner to have a separate delayed-warnings-hook and
delayed-warnings-function.  The former should be nil by default, and
should be called prior to delayed-warnings-function.

In general, it's good to avoid putting default values in hook variables.