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 09:16:20AM +0200, Ján Tomko wrote:
> 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>

Pushed, thanks.

Erik

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