The value of sort option ('S') is not used for option letter. Therefore, I'll change the single letter to non-character.
Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com> --- This patch is separated from patch of --sync option. --- cmds-qgroup.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/cmds-qgroup.c b/cmds-qgroup.c index 2a10c97..34e3bcc 100644 --- a/cmds-qgroup.c +++ b/cmds-qgroup.c @@ -313,10 +313,11 @@ static int cmd_qgroup_show(int argc, char **argv) while (1) { int c; enum { - GETOPT_VAL_SYNC = 256 + GETOPT_VAL_SORT = 256, + GETOPT_VAL_SYNC }; static const struct option long_options[] = { - {"sort", required_argument, NULL, 'S'}, + {"sort", required_argument, NULL, GETOPT_VAL_SORT}, {"sync", no_argument, NULL, GETOPT_VAL_SYNC}, { NULL, 0, NULL, 0 } }; @@ -347,7 +348,7 @@ static int cmd_qgroup_show(int argc, char **argv) case 'f': filter_flag |= 0x2; break; - case 'S': + case GETOPT_VAL_SORT: ret = btrfs_qgroup_parse_sort_string(optarg, &comparer_set); if (ret) -- 2.9.3 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html