Re: [RFC PATCH 2/4] emacs: add no-display arg to notmuch-hello-refresh-hook

Subject: Re: [RFC PATCH 2/4] emacs: add no-display arg to notmuch-hello-refresh-hook

Date: Tue, 17 Apr 2012 13:06:13 +0400

To: Jani Nikula, notmuch@notmuchmail.org

Cc:

From: Dmitry Kurochkin


Can you please give some explanaition why is this needed?  Would this
change break custom hooks that do not expect any arguments?

Regards,
  Dmitry

Jani Nikula <jani@nikula.org> writes:

> ---
>  emacs/notmuch-hello.el |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/emacs/notmuch-hello.el b/emacs/notmuch-hello.el
> index 9cd907a..0596bbe 100644
> --- a/emacs/notmuch-hello.el
> +++ b/emacs/notmuch-hello.el
> @@ -776,7 +776,7 @@ following:
>      (widget-setup)
>  
>      (goto-char final-target-pos))
> -  (run-hooks 'notmuch-hello-refresh-hook)
> +  (run-hook-with-args 'notmuch-hello-refresh-hook no-display)
>    (setq notmuch-hello-first-run nil))
>  
>  (defun notmuch-folder ()
> -- 
> 1.7.1
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

Thread: