all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: "Drew Adams" <drew.adams@oracle.com>
To: "'Chong Yidong'" <cyd@stupidchicken.com>
Cc: 2588@emacsbugs.donarmstrong.com
Subject: bug#2588: 23.0.90; Man buffer improperly formatted - wrong width
Date: Sun, 8 Mar 2009 13:16:40 -0700	[thread overview]
Message-ID: <002501c9a02a$cb2c79d0$0200a8c0@us.oracle.com> (raw)
In-Reply-To: <87y6vfomw5.fsf@cyd.mit.edu>

> > Please do not set COLUMNS to any value that is designed for frames -
> > that includes `default-frame-alist'. COLUMNS is about buffer text
> > formatting, not about frame sizing.
> >
> > The formatting width for the `Man' buffer, like that of for 
> > the *Help* buffer or the *Apropos* buffer or others, should
> > have *nothing to do* with the width of any window or frame
> > parameter - existing or default.

The point of that sentence, which your response below ignores, is that window
and frame metrics are *irrelevant* to text formatting of the `man' output.

> The `man' facility is different from apropos or help---the text is
> generated by an external formatter, not by Emacs.

So what? That doesn't change the principle. At most, it changes how the text
formatting is done - not what should be used to determine which formatting is to
be used. 

You are twisting things, here. The issue is not who (`man' vs Emacs) actually
formats the text. The issue is what should be the source for the COLUMNS value.
*Help* and *Apropos* use fixed values; they do not use a value that depend on
what they guess the displaying window or frame might be like.

> If COLUMNS is omitted, the formatter output is too long
> for the frame width, on at least one (Debian) system.

No one but you has mentioned *omitting* COLUMNS. That is a red herring. The
question is what value should be used for COLUMNS - that's all.

Where should that value come from? You don't address that. This omission
suggests you think that just because COLUMNS needs to be defined, it should be
taken from a frame parameter. That doesn't follow, at all.

> Setting COLUMNS to the frame width does the right thing most of the
> time,

Nonsense. You haven't even named one context for which it does the right thing,
let alone supported the assertion that it DTRT most of the time. (Where "most of
the time" can only mean most of the time that COLUMNS is not defined otherwise.)
I would argue that whenever `frame-width' is used here it either has a negative
effect or it has no effect. AFAICT, it never adds anything useful.

And, in any case, "most of the time" is not good enough, if the behavior is, as
it is, badly bugged for other cases.

Besides, there is absolutely no reason for this behavior, even in the cases
where it does not present a problem. It is illogical and un-Emacs to couple the
text formatting to some window or frame size.

> and covers more cases than a hardcoded value.  It's true that it
> can fail for the pop-up-frames case, and we can work around that by
> using default-frame-alist.

No, that is precisely the wrong thing to do. That continues the mistake of using
a frame parameter for text formatting decisions. Completely uncalled for.

> Certainly, it's always possible to come up
> with a special setup that makes this heuristic fail.  But if you're
> setup is so special, just set Man-width and be done with it.

This has nothing to do with my setup. Stop trying to make this sound like
something special for weird ol' Drew. I told you that my interest here is fixing
Emacs, not customizing my setup. And I mentioned that I rarely use `man' myself
nowadays. This has nothing to do with my own use of Emacs. This becomes ad
hominem argument. This is not about Drew.

You are distorting the discussion, throwing out red herrings, ignoring
straightforward arguments made by others, and trying to give the impression that
this is just a corner case - or worse, just a case of customization due to odd
personal preferences or individual setup.

Please listen: The buffer text formatting should not be based on a window or
frame measurement or setting. COLUMNS should be based on user choice (e.g.
`Man-width') or, failing that, on some buffer text-formatting setting (e.g.
`fill-column' or 80).








  parent reply	other threads:[~2009-03-08 20:16 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-07  4:22 bug#2588: 23.0.90; Man buffer improperly formatted - wrong width Chong Yidong
2009-03-07  4:50 ` Drew Adams
2009-03-07 14:15   ` Eli Zaretskii
2009-03-07 16:20     ` Drew Adams
2009-03-07 19:27       ` Eli Zaretskii
2009-03-07 19:43         ` Chong Yidong
2009-03-07 19:45           ` Drew Adams
2009-03-07 20:07           ` Eli Zaretskii
2009-03-08 16:08             ` Chong Yidong
2009-03-08 16:23               ` Drew Adams
2009-03-08 19:06                 ` Chong Yidong
2009-03-08 19:23                   ` Eli Zaretskii
2009-03-08 19:40                     ` Chong Yidong
2009-03-08 20:01                       ` Eli Zaretskii
2009-03-08 20:17                       ` Drew Adams
2009-03-09  4:05                         ` Eli Zaretskii
2009-03-09  5:33                           ` Drew Adams
2009-03-09  3:27                       ` Stefan Monnier
2009-03-09  3:38                         ` Chong Yidong
2009-03-09 13:28                           ` Stefan Monnier
2009-03-08 20:16                   ` Drew Adams [this message]
2009-03-09  4:03                     ` Eli Zaretskii
2009-03-09  5:33                       ` Drew Adams
2012-01-10 18:38                   ` Glenn Morris
2012-01-10 18:39                     ` Glenn Morris
2009-03-08 19:04               ` Eli Zaretskii
2009-03-08 19:45               ` Stefan Monnier
2009-03-08 20:04                 ` Eli Zaretskii
2011-09-11 21:51                 ` Lars Magne Ingebrigtsen
2011-09-15 18:45                   ` Juri Linkov
2011-09-17  5:22                     ` Lars Magne Ingebrigtsen
2011-10-06 22:02                   ` Lars Magne Ingebrigtsen
2011-10-07  0:34                     ` Juri Linkov
2009-03-08  4:05     ` Stefan Monnier
2009-03-08 15:45       ` Chong Yidong
2009-03-08 15:57         ` Drew Adams
2009-03-08 19:43         ` Stefan Monnier
2009-03-08 20:03           ` Eli Zaretskii
2009-03-08 20:17             ` Drew Adams
2014-07-01 23:57       ` Juri Linkov
  -- strict thread matches above, loose matches on Subject: below --
2009-03-06 20:51 Drew Adams
2009-03-07 13:58 ` Eli Zaretskii
2009-03-07 16:20   ` Drew Adams

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='002501c9a02a$cb2c79d0$0200a8c0@us.oracle.com' \
    --to=drew.adams@oracle.com \
    --cc=2588@emacsbugs.donarmstrong.com \
    --cc=cyd@stupidchicken.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.