Re: [PATCH 3/8] Remove unused and bitrotting vshCommandOptStringList

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



2010/7/8 Chris Lalancette <clalance@xxxxxxxxxx>:
> Signed-off-by: Chris Lalancette <clalance@xxxxxxxxxx>
> ---
>  tools/virsh.c |   29 -----------------------------
>  1 files changed, 0 insertions(+), 29 deletions(-)
>
> diff --git a/tools/virsh.c b/tools/virsh.c
> index d539fbc..c1451d8 100644
> --- a/tools/virsh.c
> +++ b/tools/virsh.c
> @@ -227,9 +227,6 @@ static char *vshCommandOptString(const vshCmd *cmd, const char *name,
>                                  int *found);
>  static long long vshCommandOptLongLong(const vshCmd *cmd, const char *name,
>                                        int *found);
> -#if 0
> -static int vshCommandOptStringList(const vshCmd *cmd, const char *name, char ***data);
> -#endif
>  static int vshCommandOptBool(const vshCmd *cmd, const char *name);
>
>  #define VSH_BYID     (1 << 1)
> @@ -9564,32 +9561,6 @@ vshCommandOptLongLong(const vshCmd *cmd, const char *name, int *found)
>     return res;
>  }
>
> -#if 0
> -static int
> -vshCommandOptStringList(const vshCmd *cmd, const char *name, char ***data)
> -{
> -    vshCmdOpt *arg = cmd->opts;
> -    char **val = NULL;
> -    int nval = 0;
> -
> -    while (arg) {
> -        if (arg->def && STREQ(arg->def->name, name)) {
> -            char **tmp = realloc(val, sizeof(*tmp) * (nval+1));
> -            if (!tmp) {
> -                VIR_FREE(val);
> -                return -1;
> -            }
> -            val = tmp;
> -            val[nval++] = arg->data;
> -        }
> -        arg = arg->next;
> -    }
> -
> -    *data = val;
> -    return nval;
> -}
> -#endif
> -
>  /*
>  * Returns TRUE/FALSE if the option exists
>  */
> --
> 1.7.1.1
>

ACK.

Matthias

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]