summaryrefslogtreecommitdiff
path: root/tools/vgdisplay.c
diff options
context:
space:
mode:
authorAlasdair G Kergon <agk@redhat.com>2016-06-21 22:24:52 +0100
committerAlasdair G Kergon <agk@redhat.com>2016-06-21 22:24:52 +0100
commit7e671e5dd0af699244902d5ce71a7159253c9e2b (patch)
treece14c9ff42cc60f8255a735dcae8f3e34ca1775e /tools/vgdisplay.c
parent6c269e639a121329fc035685f944816a763b23cf (diff)
downloadlvm2-7e671e5dd0af699244902d5ce71a7159253c9e2b.tar.gz
tools: Use arg_is_set instead of arg_count.
Diffstat (limited to 'tools/vgdisplay.c')
-rw-r--r--tools/vgdisplay.c36
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;