diff options
author | Alasdair G Kergon <agk@redhat.com> | 2016-06-21 22:24:52 +0100 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2016-06-21 22:24:52 +0100 |
commit | 7e671e5dd0af699244902d5ce71a7159253c9e2b (patch) | |
tree | ce14c9ff42cc60f8255a735dcae8f3e34ca1775e /tools/vgdisplay.c | |
parent | 6c269e639a121329fc035685f944816a763b23cf (diff) | |
download | lvm2-7e671e5dd0af699244902d5ce71a7159253c9e2b.tar.gz |
tools: Use arg_is_set instead of arg_count.
Diffstat (limited to 'tools/vgdisplay.c')
-rw-r--r-- | tools/vgdisplay.c | 36 |
1 files changed, 18 insertions, 18 deletions
diff --git a/tools/vgdisplay.c b/tools/vgdisplay.c index e9482df8b..7bb2e6a2b 100644 --- a/tools/vgdisplay.c +++ b/tools/vgdisplay.c @@ -19,22 +19,22 @@ static int vgdisplay_single(struct cmd_context *cmd, const char *vg_name, struct volume_group *vg, struct processing_handle *handle __attribute__((unused))) { - if (arg_count(cmd, activevolumegroups_ARG) && !lvs_in_vg_activated(vg)) + if (arg_is_set(cmd, activevolumegroups_ARG) && !lvs_in_vg_activated(vg)) return ECMD_PROCESSED; - if (arg_count(cmd, colon_ARG)) { + if (arg_is_set(cmd, colon_ARG)) { vgdisplay_colons(vg); return ECMD_PROCESSED; } - if (arg_count(cmd, short_ARG)) { + if (arg_is_set(cmd, short_ARG)) { vgdisplay_short(vg); return ECMD_PROCESSED; } vgdisplay_full(vg); /* was vg_show */ - if (arg_count(cmd, verbose_ARG)) { + if (arg_is_set(cmd, verbose_ARG)) { vgdisplay_extents(vg); process_each_lv_in_vg(cmd, vg, NULL, NULL, 0, NULL, @@ -52,30 +52,30 @@ static int vgdisplay_single(struct cmd_context *cmd, const char *vg_name, int vgdisplay(struct cmd_context *cmd, int argc, char **argv) { - if (arg_count(cmd, columns_ARG)) { - if (arg_count(cmd, colon_ARG) || - arg_count(cmd, activevolumegroups_ARG) || - arg_count(cmd, short_ARG)) { + if (arg_is_set(cmd, columns_ARG)) { + if (arg_is_set(cmd, colon_ARG) || + arg_is_set(cmd, activevolumegroups_ARG) || + arg_is_set(cmd, short_ARG)) { log_error("Incompatible options selected"); return EINVALID_CMD_LINE; } return vgs(cmd, argc, argv); - } else if (arg_count(cmd, aligned_ARG) || - arg_count(cmd, binary_ARG) || - arg_count(cmd, noheadings_ARG) || - arg_count(cmd, options_ARG) || - arg_count(cmd, separator_ARG) || - arg_count(cmd, sort_ARG) || arg_count(cmd, unbuffered_ARG)) { + } else if (arg_is_set(cmd, aligned_ARG) || + arg_is_set(cmd, binary_ARG) || + arg_is_set(cmd, noheadings_ARG) || + arg_is_set(cmd, options_ARG) || + arg_is_set(cmd, separator_ARG) || + arg_is_set(cmd, sort_ARG) || arg_is_set(cmd, unbuffered_ARG)) { log_error("Incompatible options selected"); return EINVALID_CMD_LINE; } - if (arg_count(cmd, colon_ARG) && arg_count(cmd, short_ARG)) { + if (arg_is_set(cmd, colon_ARG) && arg_is_set(cmd, short_ARG)) { log_error("Option -c is not allowed with option -s"); return EINVALID_CMD_LINE; } - if (argc && arg_count(cmd, activevolumegroups_ARG)) { + if (argc && arg_is_set(cmd, activevolumegroups_ARG)) { log_error("Option -A is not allowed with volume group names"); return EINVALID_CMD_LINE; } @@ -93,11 +93,11 @@ int vgdisplay(struct cmd_context *cmd, int argc, char **argv) vgdisplay_single); /******** FIXME Need to count number processed - Add this to process_each_vg if arg_count(cmd,activevolumegroups_ARG) ? + Add this to process_each_vg if arg_is_set(cmd,activevolumegroups_ARG) ? if (opt == argc) { log_print("no "); - if (arg_count(cmd,activevolumegroups_ARG)) + if (arg_is_set(cmd,activevolumegroups_ARG)) printf("active "); printf("volume groups found\n\n"); return LVM_E_NO_VG; |