The intention is to move vshCmddefCheckInternals out of vshCmddefOptParse to
our test suite. First step to do that is to enforce checking for an existing
help string (that also means it's non-empty) in a command because a command
without a help is not much of a use.
Signed-off-by: Erik Skultety <eskultet@xxxxxxxxxx>
---
tools/vsh.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/tools/vsh.c b/tools/vsh.c
index 4ee472c..f3b3fca 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -328,6 +328,11 @@ static int
vshCmddefCheckInternals(const vshCmdDef *cmd)
{
size_t i;
+ const char *help = NULL;
+
+ /* Each command has to provide a non-empty help string. */
+ if (!(help = vshCmddefGetInfo(cmd, "help")) || !*help)
+ return -1;
if (!cmd->opts)
return 0;
--
2.5.5
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list