[PATCH v3 04/30] docstring: variable: Move docstring from source to header file
Peter Krempa
pkrempa at redhat.com
Thu Apr 21 12:16:34 UTC 2022
On Wed, Apr 20, 2022 at 21:07:53 +0200, Victor Toso wrote:
> virConnectAuthPtrDefault is the only variable that we export and
> with the documentation being at the C file, it makes harder to find
> it.
>
> With this patch, the documentation of this variable can be found by
> those who look at the header file for guidance plus, it will also be
> included in the XML API, as the followup patch will fix apibuild.py
> to store what is documented in the header file for this variable.
>
> Note that the apibuild, does not do the same with .c files. It'd
> require adding it for (parsing) recursion, similar to what is done
> for functions. Considering the pros/cons, seems more reasonable to
> move the docstring to the header file instead.
>
> Signed-off-by: Victor Toso <victortoso at redhat.com>
> ---
> include/libvirt/libvirt-host.h | 13 +++++++++++++
> src/libvirt.c | 11 +----------
> 2 files changed, 14 insertions(+), 10 deletions(-)
>
> diff --git a/include/libvirt/libvirt-host.h b/include/libvirt/libvirt-host.h
> index 1dc4b8a147..8cccf2921f 100644
> --- a/include/libvirt/libvirt-host.h
> +++ b/include/libvirt/libvirt-host.h
> @@ -562,6 +562,19 @@ struct _virConnectAuth {
> typedef struct _virConnectAuth virConnectAuth;
> typedef virConnectAuth *virConnectAuthPtr;
>
> +/*
> + * virConnectAuthPtrDefault:
> + *
> + * A default implementation of the authentication callbacks. This
> + * implementation is suitable for command line based tools. It will
> + * prompt for username, passwords, realm and one time keys as needed.
> + * It will print on STDOUT, and read from STDIN. If this is not
> + * suitable for the application's needs an alternative implementation
> + * should be provided.
> + *
> + * Since: v0.4.1
This one is present in v0.4.0, but the tag is missing in the repository.
I'll change this and push it later.
Reviewed-by: Peter Krempa <pkrempa at redhat.com>
More information about the libvir-list
mailing list