Eric Blake wrote:
> Ondřej Vašík <ovasik <at> redhat.com> writes:
>> Good, point. Added blurb about the environment variables to all of those
>> utilities.
>
>>        fputs (_("\n\
>> +If none of the environment variables BLOCKSIZE, BLOCK_SIZE or \n\
>> +<COMMAND>_BLOCKSIZE is specified, blocksize defaults to 1024 bytes.\n\
>
> When I do du --help, I'd rather see this spelled out as DU_BLOCKSIZE, not
> <COMMAND>_BLOCKSIZE.
>
>> +(or 512 bytes when POSIXLY_CORRECT environment variable is set)\n\
>> +"), stdout);
>
> This repetition begs for a helper method in system.h.  It would look better
> like:
>
> emit_blocksize (program_name);

I agree.
Though please call it something like "emit_blocksize_note"
so it's name doesn't imply it's printing a bare number.

> where emit_blocksize takes care of converting program_name to uppercase in its
> spelling of %s_BLOCKSIZE.
>
> But at least with this version of the patch, you are adding an entirely new
> string for the translators, rather than modifying an existing one, which means
> you won't be breaking any translations, and we have less reason to exclude it
> from 7.6 if Jim likes the idea.

I do indeed.


Reply via email to