unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: "積丹尼 Dan Jacobson" <jidanni@jidanni.org>
Cc: larsi@gnus.org, 40888@debbugs.gnu.org, stefan@marxist.se,
	tomasn@posteo.net
Subject: bug#40888: M-x man: don't redraw good pages when not finding bad pages
Date: Thu, 06 Aug 2020 16:40:48 +0300	[thread overview]
Message-ID: <83o8nn99jz.fsf@gnu.org> (raw)
In-Reply-To: <877duc9w58.5.fsf@jidanni.org>

> From: 積丹尼 Dan Jacobson
>  <jidanni@jidanni.org>
> Date: Thu, 06 Aug 2020 13:32:51 +0800
> Cc: Tomas Nordin <tomasn@posteo.net>, 40888@debbugs.gnu.org,
>  Stefan Kangas <stefan@marxist.se>
> 
> Well, just like we don't purchase a house for our client before we check
> if he has money in the bank, even if it means waiting for business
> hours, it seems the program should first make sure what it intends to
> display even exists. "test -f" certainly won't slow things down that much.

"test -f" won't do the job, because 'man' employs a non-trivial logic
to find the man page, using environment variables, command-line
options, and hardcoded directories and file names.  IOW, there's no
easy way of knowing what to put after "test -f".

Sounds like you are suggesting that Emacs either (a) reproduces all
that 'man' logic internally (not easily done, as different
implementations vary in how they do it), or (b)rely on 'man' itself
telling whether the file exists, which AFAIK must use command-line
options that aren't necessarily available in all versions of 'man', to
say nothing about slowing down the command due to duplicate
invocation.

All that to solve a minor annoyance?  Is that really justified?





  parent reply	other threads:[~2020-08-06 13:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-26 22:59 bug#40888: M-x man: don't redraw good pages when not finding bad pages 積丹尼 Dan Jacobson
2020-04-27  8:33 ` Tomas Nordin
2020-04-27 14:54   ` 積丹尼 Dan Jacobson
2020-04-27 15:07     ` Stefan Kangas
2020-04-27 20:25       ` Tomas Nordin
2020-08-05 11:28       ` Lars Ingebrigtsen
2020-08-06  5:32         ` 積丹尼 Dan Jacobson
2020-08-06  7:17           ` Lars Ingebrigtsen
2020-08-06 13:40           ` Eli Zaretskii [this message]
2020-08-06 17:39             ` 積丹尼 Dan Jacobson
2020-08-06 17:41               ` Lars Ingebrigtsen
2022-02-20 14:17                 ` Lars Ingebrigtsen

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=83o8nn99jz.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=40888@debbugs.gnu.org \
    --cc=jidanni@jidanni.org \
    --cc=larsi@gnus.org \
    --cc=stefan@marxist.se \
    --cc=tomasn@posteo.net \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).