shaba/lvm2
1
0
Fork 0
mirror of git://sourceware.org/git/lvm2.git synced 2025-01-03 05:18:29 +03:00
Code Releases Activity

man/help: improve the style of units indicator

Browse Source 
We use --foo Number[k|Units] to indicate that
the default units of the number is k, but other
units listed below are also accepted.

Capitalize and underline Units so it is consistent
with other variables, and reference it at the end.

Technically, the k should be bold, but this
tends to make the text visually hard to read
because of the excessive highlights scattered
everywhere.  So it's left normal text for now
(it's unlikely to confuse anyone.)
This commit is contained in:
David Teigland 2017-02-22 16:50:30 -06:00
parent da634bfc89
commit 3f4ecaf8c2
2 changed files with 15 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
tools/command.c
View File

@ -1786,20 +1786,19 @@ static void print_val_man(const char *str)
int i;
/*
* The suffix [k|unit] is just printed in plain text.
* Doing bold k and underlined unit creates a lot of
* visual "noise" that is choppy and hard to read.
* Doing bold k before underlined Unit creates a lot of
* visual "noise" that makes the text hard to read.
* The extra markup in this case doesn't add anything
* that isn't already obvious.
*/
if (!strcmp(str, "Number[k|unit]")) {
printf("\\fINumber\\fP[k|unit]");
if (!strcmp(str, "Number[k|Unit]")) {
printf("\\fINumber\\fP[k|\\fIUnit\\fP]");
return;
}
if (!strcmp(str, "Number[m|unit]")) {
printf("\\fINumber\\fP[m|unit]");
if (!strcmp(str, "Number[m|Unit]")) {
printf("\\fINumber\\fP[m|\\fIUnit\\fP]");
return;
}
@ -2799,13 +2798,13 @@ void print_man_all_positions_desc(struct command_name *cname)
/* Nearly every command uses a number arg somewhere. */
printf("\n.HP\n");
printf("\\fINumber\\fP");
printf("\\fINumber\\fP, \\fIUnit\\fP");
printf("\n");
printf(".br\n");
printf("Input units are always treated as base two values, regardless of unit\n"
"capitalization, e.g. 'k' and 'K' both refer to 1024.\n"
"The default input unit is specified by letter, followed by |unit which\n"
"represents other possible input units: bBsSkKmMgGtTpPeE.\n");
"The default input unit is specified by letter, followed by |\\fIUnit\\fP\n"
"which represents other possible input units: \\fBbBsSkKmMgGtTpPeE\\fP.\n");
printf("\n.HP\n");
printf("Environment");

12
tools/vals.h
View File

@ -83,10 +83,10 @@
* could be "Number[bBsSkKmMgGtTpPeE]" (with implied |),
* but repeating this full specification produces cluttered
* output, and doesn't indicate which unit is the default.
* "Number[units]" would be cleaner, as would a subset of
* "Number[Units]" would be cleaner, as would a subset of
* common units, e.g. "Number[kmg...]", but neither helps
* with default. "Number[k|unit]" and "Number[m|unit]" show
* the default, and "unit" indicates that other units
* with default. "Number[k|Unit]" and "Number[m|Unit]" show
* the default, and "Unit" indicates that other units
* are possible without listing them all. This also
* suggests using the preferred lower case letters, because
* --size and other option args treat upper/lower letters
@ -115,9 +115,9 @@ val(activation_VAL, activation_arg, "Active", "y|n|ay")
val(cachemode_VAL, cachemode_arg, "CacheMode", "writethrough|writeback")
val(discards_VAL, discards_arg, "Discards", "passdown|nopassdown|ignore")
val(mirrorlog_VAL, mirrorlog_arg, "MirrorLog", "core|disk")
val(sizekb_VAL, size_kb_arg, "SizeKB", "Number[k|unit]")
val(sizemb_VAL, size_mb_arg, "SizeMB", "Number[m|unit]")
val(regionsize_VAL, regionsize_arg, "RegionSize", "Number[m|unit]")
val(sizekb_VAL, size_kb_arg, "SizeKB", "Number[k|Unit]")
val(sizemb_VAL, size_mb_arg, "SizeMB", "Number[m|Unit]")
val(regionsize_VAL, regionsize_arg, "RegionSize", "Number[m|Unit]")
val(numsigned_VAL, int_arg_with_sign, "SNumber", "[+|-]Number")
val(numsignedper_VAL, int_arg_with_sign_and_percent, "SNumberP", "[+|-]Number[%VG|%PVS|%FREE]")
val(permission_VAL, permission_arg, "Permission", "rw|r")