On 16/10/2019 16:31, Nikolay Borisov wrote:
[...]
>> + printf("\tfsck successfully repair all types of filesystem corruption. Eg.\n");
>> + printf("\tsome other software or hardware bugs can fatally damage a volume.\n");
>
> nit: The word 'other' here is redundant, no ?
Hmm really, maybe. But the sentence above lacks a 'can' (I'll fix it up
in the manpage as well.
>
>> + printf("\tThe operation will start in %d seconds.\n", delay);
>> + printf("\tUse Ctrl-C to stop it.\n");
>> + while (delay) {
>> + printf("%2d", delay--);
>> + fflush(stdout);
>> + sleep(1);
>> + }
>
> That's a long winded way to have a simple for loop that prints 10 dots,
> 1 second apart. IMO a better use experience would be to ask the user to
> confirm and if the '-f' options i passed don't bother printing the
> warning at all.
That's just copy & paste from cmds/balance.c
--
Johannes Thumshirn SUSE Labs Filesystems
jthumshirn@xxxxxxx +49 911 74053 689
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5
90409 Nürnberg
Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Felix Imendörffer
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
