[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] doc: du - clarify default blocksize in usage/manpage
From: |
Jim Meyering |
Subject: |
Re: [PATCH] doc: du - clarify default blocksize in usage/manpage |
Date: |
Thu, 10 Sep 2009 15:19:42 +0200 |
Pádraig Brady wrote:
> While this patch is more verbose than my previous patch
> in this thread, it is more accurate. So let's drop the
> previous one. However...
>
> The info was inaccurate for ls, so I've split the SIZE
> specific info into a new emit_size_note(). That also
> allows refactoring to be done in split and truncate.
>
> I've also changed the text in emit_blocksize_note() a bit.
>
> Also I sync'd the SIZE text used in the info docs,
> which is now more accurate.
>
> cheers,
> Pádraig
>>From 43ee481910b5949941a6f7724c754c646875488f Mon Sep 17 00:00:00 2001
> From: =?utf-8?q?Ond=C5=99ej=20Va=C5=A1=C3=ADk?= <address@hidden>
> Date: Thu, 10 Sep 2009 12:33:41 +0100
> Subject: [PATCH] doc: improve various BLOCKSIZE and SIZE help
>
> * doc/coreutils.texi (multiplierSuffixes): Mention that
> the suffix can be specified without a leading number
> * src/split.c (usage): Refactor SIZE help to within a function
> * src/truncate.c (usage): Likewise
> * src/ls.c (usage): Likewise
> * src/df.c (usage): Likewise. Also add a function with BLOCKSIZE help
> * src/du.c (usage): Likewise.
> * src/system.h: Define 2 functions to emit common help text
> This was prompted by https://bugzilla.redhat.com/show_bug.cgi?id=511188
...
Looks good modulo two indenatation nits:
Please commit.
diff --git a/src/system.h b/src/system.h
index 878a001..7cdad4a 100644
--- a/src/system.h
+++ b/src/system.h
@@ -619,7 +619,7 @@ ptr_align (void const *ptr, size_t alignment)
static inline void
emit_size_note (void)
{
- fputs (_("\n\
+ fputs (_("\n\
SIZE may be (or may be an integer optionally followed by) one of following:\n\
KB 1000, K 1024, MB 1000*1000, M 1024*1024, and so on for G, T, P, E, Z, Y.\n\
"), stdout);
@@ -628,7 +628,7 @@ KB 1000, K 1024, MB 1000*1000, M 1024*1024, and so on for
G, T, P, E, Z, Y.\n\
static inline void
emit_blocksize_note (char const *program)
{
- printf (_("\n\
+ printf (_("\n\
If none of the environment variables BLOCKSIZE, BLOCK_SIZE or %s_BLOCK_SIZE\n\
or --block-size option specify SIZE, display in units of 1024 bytes.\n\
(or 512 bytes if the POSIXLY_CORRECT environment variable is set)\n\
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, (continued)
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Eric Blake, 2009/09/09
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Jim Meyering, 2009/09/09
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Ondřej Vašík, 2009/09/09
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Pádraig Brady, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Eric Blake, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Pádraig Brady, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Jim Meyering, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Pádraig Brady, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Jim Meyering, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Jim Meyering, 2009/09/10
- Re: [PATCH] doc: du - clarify default blocksize in usage/manpage,
Jim Meyering <=
Re: [PATCH] doc: du - clarify default blocksize in usage/manpage, Pádraig Brady, 2009/09/09