linux-man.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michael Kerrisk (man-pages)" <mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] proc(5): fix indentation for net examples
Date: Thu, 02 Jan 2014 08:48:25 +1300	[thread overview]
Message-ID: <52C47109.2000708@gmail.com> (raw)
In-Reply-To: <201312310043.48914.vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>

On 12/31/13 18:43, Mike Frysinger wrote:
> On Monday 30 December 2013 06:27:19 Michael Kerrisk (man-pages) wrote:
>> On 12/29/13 02:26, Mike Frysinger wrote:
>>> Signed-off-by: Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
>>
>> I'm not sure what the goal of this change is. What's broken?
>> (Note that there is some value, in terms of easy parsing of
>> man output, to having only the section headings start in
>> column 1.)
> 
> i'm not sure what you mean here.  having it at col 1 makes scanning the man 
> page harder imo, and indenting it to match the content that it's part of makes 
> sense.  it's what most every other section does in this man page already.
> 
> if you're worried about 80 cols, then this table already widely exceeds that 
> limit even when placed at col 1.  so fixing the indentation doesn't really hurt 
> that scenario.
> -mike

I misread your patch somewhat. However, I prefer to keep things the way they
are, for a couple of reasons. First, these sort of adjustments do allow some
of the other tables to fit in 80 columns where they would not otherwise do so.
Second, they reduce the min. width of the terminal window that is required 
to properly display the tables.

Cheers,

Michael


-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      parent reply	other threads:[~2014-01-01 19:48 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-28 13:26 [PATCH] proc(5): fix indentation for net examples Mike Frysinger
     [not found] ` <1388237204-12791-1-git-send-email-vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
2013-12-30 11:27   ` Michael Kerrisk (man-pages)
     [not found]     ` <52C15897.5060309-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2013-12-31  5:43       ` Mike Frysinger
     [not found]         ` <201312310043.48914.vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
2014-01-01 19:48           ` Michael Kerrisk (man-pages) [this message]

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=52C47109.2000708@gmail.com \
    --to=mtk.manpages-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org \
    /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.
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 NNTP newsgroup(s).