On Sun, Apr 09, 2017 at 02:58:44PM -0400, Derek Schrock wrote:
> Add option $beep_new_buffy that controls if mutt should beep for buffy 
> notifications.

Ordinarily I'm not a fan of "micro-options" like this.  However, beeps
annoy me to the extent that I can understand the desire to control when
they occur more precisely.

One concern is that the behavior of $beep_new is now less clear unless
you look at the code.  I think its documentation should be revised to
state that it will beep when new mail arrives in the currently open
mailbox.

My other concern is whether this new option should be predicated on the
$beep_new option.  Does it make sense to want to beep for buffy
notifications, but not for the currently open mailbox?

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA

Attachment: signature.asc
Description: PGP signature

Reply via email to