diff options
author | David Teigland <teigland@redhat.com> | 2017-02-07 15:12:24 -0600 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2017-02-13 08:21:58 -0600 |
commit | 13a6368522894e9d32544d8fb3d6360bbe9018d1 (patch) | |
tree | b9cc5022429edd30b62dd47ed0b56b598fa93567 /tools/vals.h | |
parent | a1386dcca024b51019c1f3be6b153249df06dbac (diff) | |
download | lvm2-13a6368522894e9d32544d8fb3d6360bbe9018d1.tar.gz |
args: use arg parsing function for region size
Consolidate the validation of the region size arg
in a new arg parsing function.
Diffstat (limited to 'tools/vals.h')
-rw-r--r-- | tools/vals.h | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/tools/vals.h b/tools/vals.h index 12d315f80..38029aadc 100644 --- a/tools/vals.h +++ b/tools/vals.h @@ -92,6 +92,10 @@ * --size and other option args treat upper/lower letters * the same, all as 1024 SI base. For this reason, we * should avoid suggesting the upper case letters. + * + * FIXME: negative numbers should be automatically rejected + * for anything but int_arg_with_sign(), e.g. + * size_mb_arg() should reject a negative number. */ val(none_VAL, NULL, "None", "ERR") /* unused, for enum value 0 */ @@ -113,6 +117,7 @@ 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(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") |