mirror of
https://gitlab.com/libvirt/libvirt.git
synced 2024-12-23 14:15:28 +00:00
3b4df5d350
In few places we have the following code pattern: int ret; ... /* @ret is not accessed here */ ret = f(...); return ret; This pattern can be written less verbose: ... return f(...); This patch was generated with following coccinelle spatch: @@ type T; constant C; expression f; identifier ret; @@ -T ret = C; ... when != ret -ret = f; -return ret; +return f; Afterwards I needed to fix a few places, e.g. comment in virDomainNetIPParseXML() was removed too because coccinelle thinks it refers to @ret while in fact it doesn't. Also in few places it replaced @ret declaration with a few spaces instead of removing the line. But nothing terribly wrong. Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>
111 lines
3.0 KiB
C
111 lines
3.0 KiB
C
/*
|
|
* virsh-completer-nodedev.c: virsh completer callbacks related to nodedev
|
|
*
|
|
* Copyright (C) 2019 Red Hat, Inc.
|
|
*
|
|
* This library is free software; you can redistribute it and/or
|
|
* modify it under the terms of the GNU Lesser General Public
|
|
* License as published by the Free Software Foundation; either
|
|
* version 2.1 of the License, or (at your option) any later version.
|
|
*
|
|
* This library is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
* Lesser General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU Lesser General Public
|
|
* License along with this library. If not, see
|
|
* <http://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
#include <config.h>
|
|
|
|
#include "virsh-completer-nodedev.h"
|
|
#include "conf/node_device_conf.h"
|
|
#include "viralloc.h"
|
|
#include "virsh-nodedev.h"
|
|
#include "virsh.h"
|
|
#include "virstring.h"
|
|
|
|
char **
|
|
virshNodeDeviceNameCompleter(vshControl *ctl,
|
|
const vshCmd *cmd G_GNUC_UNUSED,
|
|
unsigned int flags)
|
|
{
|
|
virshControlPtr priv = ctl->privData;
|
|
virNodeDevicePtr *devs = NULL;
|
|
int ndevs = 0;
|
|
size_t i = 0;
|
|
char **ret = NULL;
|
|
VIR_AUTOSTRINGLIST tmp = NULL;
|
|
|
|
virCheckFlags(0, NULL);
|
|
|
|
if (!priv->conn || virConnectIsAlive(priv->conn) <= 0)
|
|
return NULL;
|
|
|
|
if ((ndevs = virConnectListAllNodeDevices(priv->conn, &devs, flags)) < 0)
|
|
return NULL;
|
|
|
|
if (VIR_ALLOC_N(tmp, ndevs + 1) < 0)
|
|
goto cleanup;
|
|
|
|
for (i = 0; i < ndevs; i++) {
|
|
const char *name = virNodeDeviceGetName(devs[i]);
|
|
|
|
tmp[i] = g_strdup(name);
|
|
}
|
|
|
|
ret = g_steal_pointer(&tmp);
|
|
|
|
cleanup:
|
|
for (i = 0; i < ndevs; i++)
|
|
virNodeDeviceFree(devs[i]);
|
|
VIR_FREE(devs);
|
|
return ret;
|
|
}
|
|
|
|
|
|
char **
|
|
virshNodeDeviceEventNameCompleter(vshControl *ctl G_GNUC_UNUSED,
|
|
const vshCmd *cmd G_GNUC_UNUSED,
|
|
unsigned int flags)
|
|
{
|
|
size_t i = 0;
|
|
VIR_AUTOSTRINGLIST tmp = NULL;
|
|
|
|
virCheckFlags(0, NULL);
|
|
|
|
if (VIR_ALLOC_N(tmp, VIR_NODE_DEVICE_EVENT_ID_LAST + 1) < 0)
|
|
return NULL;
|
|
|
|
for (i = 0; i < VIR_NODE_DEVICE_EVENT_ID_LAST; i++)
|
|
tmp[i] = g_strdup(virshNodeDeviceEventCallbacks[i].name);
|
|
|
|
return g_steal_pointer(&tmp);
|
|
}
|
|
|
|
|
|
char **
|
|
virshNodeDeviceCapabilityNameCompleter(vshControl *ctl,
|
|
const vshCmd *cmd,
|
|
unsigned int flags)
|
|
{
|
|
VIR_AUTOSTRINGLIST tmp = NULL;
|
|
const char *cap_str = NULL;
|
|
size_t i = 0;
|
|
|
|
virCheckFlags(0, NULL);
|
|
|
|
if (vshCommandOptStringQuiet(ctl, cmd, "cap", &cap_str) < 0)
|
|
return NULL;
|
|
|
|
if (VIR_ALLOC_N(tmp, VIR_NODE_DEV_CAP_LAST + 1) < 0)
|
|
return NULL;
|
|
|
|
for (i = 0; i < VIR_NODE_DEV_CAP_LAST; i++)
|
|
tmp[i] = g_strdup(virNodeDevCapTypeToString(i));
|
|
|
|
return virshCommaStringListComplete(cap_str, (const char **)tmp);
|
|
}
|