From c02736059a8f3c7966b2e2796a43e8c438f87ccc Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Wed, 21 Mar 2018 17:17:54 +0100 Subject: [PATCH] virCommandPassFD: Give name to flags The flags passed to virCommandPassFD() are unnamed and documentation to this function doesn't list them either. Give them name and mention it in documentation to functions using them. Signed-off-by: Michal Privoznik Reviewed-by: Laine Stump --- src/util/vircommand.c | 8 ++++---- src/util/vircommand.h | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/util/vircommand.c b/src/util/vircommand.c index 17cbdab7b7..8a319069fd 100644 --- a/src/util/vircommand.c +++ b/src/util/vircommand.c @@ -173,9 +173,9 @@ virCommandFDIsSet(virCommandPtr cmd, /* * virCommandFDSet: - * @fd: FD to be put into @set - * @set: the set - * @set_size: actual size of @set + * @cmd: pointer to virCommand + * @fd: file descriptor to pass + * @flags: extra flags; binary-OR of virCommandPassFDFlags * * This is practically generalized implementation * of FD_SET() as we do not want to be limited @@ -976,7 +976,7 @@ virCommandNewVAList(const char *binary, va_list list) * virCommandPassFD: * @cmd: the command to modify * @fd: fd to reassign to the child - * @flags: the flags + * @flags: extra flags; binary-OR of virCommandPassFDFlags * * Transfer the specified file descriptor to the child, instead * of closing it on exec. @fd must not be one of the three diff --git a/src/util/vircommand.h b/src/util/vircommand.h index d59278cf5f..883e212959 100644 --- a/src/util/vircommand.h +++ b/src/util/vircommand.h @@ -51,10 +51,10 @@ virCommandPtr virCommandNewVAList(const char *binary, va_list list) * delayed until the Run/RunAsync methods */ -enum { +typedef enum { /* Close the FD in the parent */ VIR_COMMAND_PASS_FD_CLOSE_PARENT = (1 << 0), -}; +} virCommandPassFDFlags; void virCommandPassFD(virCommandPtr cmd, int fd, -- GitLab