[PATCH v3 6/7] virdnsmasq: Drop dnsmasqCapsNewFromBuffer()
Andrea Bolognani
abologna at redhat.com
Fri Jan 14 17:01:34 UTC 2022
On Wed, Jan 12, 2022 at 09:47:57AM +0100, Michal Privoznik wrote:
> The function is no longer used. Remove it.
>
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
> src/libvirt_private.syms | 1 -
> src/util/virdnsmasq.c | 14 --------------
> src/util/virdnsmasq.h | 1 -
> 3 files changed, 16 deletions(-)
Reviewed-by: Andrea Bolognani <abologna at redhat.com>
Further cleanup opportunities:
* drop the 'force' argument of dnsmasqCapsRefreshInternal(), as the
only caller always passes true;
* drop the 'noRefresh' field of the dnsmasqCaps struct, which
AFAICT doesn't really do anything even before your changes;
* rework dnsmasqCapsSetFromBuffer() so that it takes the output of
the two calls to dnsmasq as separate arguments, instead of doing
the silly dance of joining the two in the caller only to have the
function split them again immediately.
--
Andrea Bolognani / Red Hat / Virtualization
More information about the libvir-list
mailing list