Re: [libvirt] [PATCH 2/2] Fix the build on non-linux platforms after VIR_AUTOPTR related changes
On Wed, Aug 08, 2018 at 08:23:42AM +0200, Erik Skultety wrote:
Commits 7b706f33ac and 4acb7887e4 introduced some compound type *Free
wrappers in order to use them with VIR_DEFINE_AUTOPTR_FUNC. However,
since those were not used in the code right away, Clang complained about
unused functions (static ones that are defined by the macro above).
This patch puts the defined functions in use.
If you keep the series bisectable even with clang with warnings turned
on, you can GET a cookie!
Signed-off-by: Erik Skultety <eskultet(a)redhat.com>
---
src/util/virnetdev.c | 3 +--
src/util/virnetlink.c | 6 ++----
2 files changed, 3 insertions(+), 6 deletions(-)
This fixes the build for me with CLang 5.0.1
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano
Attachments:
- signature.asc (application/pgp-signature — 833 bytes)