Subject: Re: bug#18133: Suppressing asynchronous command output

From: Eli Zaretskii <[email-masked]>
Date: Tue, 27 Dec 2016 08:23:31 +0200

> From: Reuben Thomas <[email-masked]>
> Date: Tue, 27 Dec 2016 01:20:42 +0000
> Cc: martin rudalics <[email-masked]>, [email-masked], Eli Zaretskii <[email-masked]>
> 
> +(defun comint-make-newly-written-buffer-visible (string process)
> +  "Make the output buffer visible when output is added to an empty buffer.
> +Useful in `comint-preoutput-filter-functions'."
> +  (let ((buffer (process-buffer process)))
> +    (when (and (= 0 (buffer-size buffer))
> +               (string-match-p "\\*Async Shell Command\\*"
> +                               (buffer-name buffer)))
> +      (display-buffer (process-buffer process))))
> +  string)

Why are we hard-coding a certain buffer name in a function that is
supposed to be more general, judging by its name and doc string?