Re: [PATCH 2/2] Fix the build on non-linux platforms after VIR_AUTOPTR related changes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
---
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@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: Digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux