On 4/20/13 4:13 PM, Frank A. Kingswood wrote:
> On 04/03/13 01:37, [email protected] wrote:
>> From: Zhi Yong Wu<[email protected]>
>>
>>    Since raid5/6 support was introduced, we should update mkfs.btrfs help 
>> info.
>>
>> Signed-off-by: Zhi Yong Wu<[email protected]>
>> ---
>>   mkfs.c | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/mkfs.c b/mkfs.c
>> index 5ece186..f9f26a5 100644
>> --- a/mkfs.c
>> +++ b/mkfs.c
>> @@ -326,7 +326,7 @@ static void print_usage(void)
>>       fprintf(stderr, "options:\n");
>>       fprintf(stderr, "\t -A --alloc-start the offset to start the FS\n");
>>       fprintf(stderr, "\t -b --byte-count total number of bytes in the 
>> FS\n");
>> -    fprintf(stderr, "\t -d --data data profile, raid0, raid1, raid10, dup 
>> or single\n");
>> +    fprintf(stderr, "\t -d --data data profile, raid0, raid1, raid5, raid6, 
>> raid10, dup or single\n");
>>       fprintf(stderr, "\t -l --leafsize size of btree leaves\n");
>>       fprintf(stderr, "\t -L --label set a label\n");
>>       fprintf(stderr, "\t -m --metadata metadata profile, values like data 
>> profile\n");
> 
> Nitpick: that line of output is now longer than 80. Perhaps instead of "\t " 
> have a few spaces at the start of each line.

or maybe:

+    fprintf(stderr, "\t -d --data data profile: 
[raid0|raid1|raid5|raid6|raid10|dup|single]\n");

I think that fits...

> Frank
> 
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
> the body of a message to [email protected]
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to