Re: [PATCH 2/2] virsh: Add coredump format completion to dump command

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

 



On 6/7/21 5:14 AM, Lin Ma wrote:
> Signed-off-by: Lin Ma <lma@xxxxxxxx>
> ---
>  tools/virsh-completer-domain.c | 19 +++++++++++++++++++
>  tools/virsh-completer-domain.h |  5 +++++
>  tools/virsh-domain.c           |  2 ++
>  3 files changed, 26 insertions(+)
> 

> diff --git a/tools/virsh-completer-domain.h b/tools/virsh-completer-domain.h
> index ef242d0c68..f548f7ba9e 100644
> --- a/tools/virsh-completer-domain.h
> +++ b/tools/virsh-completer-domain.h
> @@ -122,3 +122,8 @@ char ** virshKeycodeNameCompleter(vshControl *ctl,
>  char ** virshDomainFSMountpointsCompleter(vshControl *ctl,
>                                            const vshCmd *cmd,
>                                            unsigned int flags);
> +
> +char **
> +virshDomainCoreDumpFormatCompleter(vshControl *ctl G_GNUC_UNUSED,
> +                                   const vshCmd *cmd G_GNUC_UNUSED,
> +                                   unsigned int flags);

We don't like marking arguments as unused in header files. That's left
for corresponding .c file to do.

Michal




[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]

  Powered by Linux