all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Chris Siebenmann <cks.emacsbugs-01@cs.toronto.edu>
To: Mike Kupfer <kupfer@rawbw.com>
Cc: Chris Siebenmann <cks.emacsbugs-01@cs.toronto.edu>,
	Eli Zaretskii <eliz@gnu.org>, Bill Wohler <wohler@newt.com>,
	67359@debbugs.gnu.org
Subject: bug#67359: 29.1; 29.1: MH-E limited display malfunctions if nothing is matched
Date: Wed, 22 Nov 2023 11:21:29 -0500	[thread overview]
Message-ID: <3289159.1700670089@apps0.cs.toronto.edu> (raw)
In-Reply-To: <9058.1700669328@alto>

> > > From: Chris Siebenmann <cks.emacsbugs-01@cs.toronto.edu>
> > > Date: Tue, 21 Nov 2023 19:28:45 -0500
> [...]
> > > When you do this, MH-E will first show an error message in a buffer
> > > below the folder window and then narrow the main folder window to a
> > > single line showing 'scan: no messages match specification'. ('/ w' will
> > > then fix the situation.)
>
> Yes, I can reproduce this.  
>
> Chris, I'm not sure what you're expecting instead.  I'm guessing you'd
> like to have the original message list in the folder buffer (no need to
> type "/ w"), with the error text displayed like a normal error, rather
> than in the folder buffer.  Do I understand correctly?

I think the folder should keep the original message list. I don't know
what should be done with the error message, since it's semi-redundant. I
say this because the current limit-to code appears to contain an attempt
to handle the case where no messages match. mh-narrow-to-header-field
has, at the end:

	(if (null msg-list)
	    (message "No matches")
	  [... regular narrowing code ...] )

but since its parsing of pick output puts 0's in msg-list instead of
leaving it empty here, you never get the 'No matches' message. So I
think the earlier code should arrange to leave msg-list empty by not
adding '0' message numbers to it (they're invalid anyway), and then this
code can act normally and people get the 'No matches' message and have
nothing happen.

On the one hand, showing pick's error message for this is redundant with
the 'No matches' message. On the other hand, there might be other pick
errors in odder cases that one wouldn't want to throw away.

(I ran into this issue because I wrote some general pick-based
limit-to-search functions for my own use, and sometimes what I was
checking for wasn't there; in that case I definitely want to leave the
message list intact. Many people probably use the / stuff without a
prefix argument to modify the search terms, or only search for things
that are there in at least one message.)

	- cks





  reply	other threads:[~2023-11-22 16:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-11-22  0:28 bug#67359: 29.1; 29.1: MH-E limited display malfunctions if nothing is matched Chris Siebenmann
2023-11-22 14:43 ` Eli Zaretskii
2023-11-22 16:08   ` Mike Kupfer
2023-11-22 16:21     ` Chris Siebenmann [this message]
2023-11-23  1:56   ` Bill Wohler via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-11-23  6:53     ` Eli Zaretskii
2023-11-23 19:59       ` Mike Kupfer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3289159.1700670089@apps0.cs.toronto.edu \
    --to=cks.emacsbugs-01@cs.toronto.edu \
    --cc=67359@debbugs.gnu.org \
    --cc=eliz@gnu.org \
    --cc=kupfer@rawbw.com \
    --cc=wohler@newt.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.