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
prev 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).