From: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
---|---|
To: | tgl(at)sss(dot)pgh(dot)pa(dot)us |
Cc: | shigeru(dot)hanada(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: missing description initdb manual |
Date: | 2012-04-15 00:21:56 |
Message-ID: | 20120415.092156.2129817367807325887.t-ishii@sraoss.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
> I think this line needs some more thought:
>
>> - printf(_(" -m SHUTDOWN-MODE can be \"smart\", \"fast\", or \"immediate\"\n"));
>> + printf(_(" -m, --mode SHUTDOWN-MODE can be \"smart\", \"fast\", or \"immediate\"\n"));
>
> because it's not respecting the intended alignment of the explanation
> column. (It wasn't before, either, but that's not an excuse to not fix
> it.) I think you could make it read just "-m, --mode MODE" and then
> there would be room to align this switch's explanation with the ones
> above it. I'm tempted to also say that the explanation should be
> MODE can be \"smart\", \"fast\", or \"immediate\"
Thanks for the suggestion. I modified the part as you suggested and
committed the changes.
--
Tatsuo Ishii
SRA OSS, Inc. Japan
English: http://www.sraoss.co.jp/index_en.php
Japanese: http://www.sraoss.co.jp
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2012-04-15 02:33:13 | Re: Patch: add timing of buffer I/O requests |
Previous Message | Robert Haas | 2012-04-15 00:12:00 | Re: Patch: add timing of buffer I/O requests |