[libvirt] [PATCH 1/1] admin_server: use VIR_AUTOFREE() in adminClientGetInfo string

Daniel P. Berrangé berrange at redhat.com
Tue Oct 1 14:54:02 UTC 2019


On Tue, Oct 01, 2019 at 11:49:25AM -0300, Daniel Henrique Barboza wrote:
> 
> 
> On 10/1/19 11:25 AM, Ján Tomko wrote:
> > On Tue, Oct 01, 2019 at 03:59:07PM +0200, Erik Skultety wrote:
> > > On Tue, Oct 01, 2019 at 09:12:58AM -0300, Daniel Henrique Barboza wrote:
> > > > Signed-off-by: Daniel Henrique Barboza <danielhb413 at gmail.com>
> > > > ---
> > > Reviewed-by: Erik Skultety <eskultet at redhat.com>
> > > 
> > > and safe for freeze
> > > 
> > 
> > This is not a bugfix, it can wait.
> > 
> > Also, there is an ongoing discussion about using g_autofree once we
> > start depending on gnulib, so I don't see a point in trying to convert
> > code to the macro that will soon be obsolete.
> 
> Huh. Guess I'll stop sending those VIR_AUTO* patches all around then. I'll
> wait for the gnulib equivalent, if there will be any.

NB, glib != gnulib :-)

gnulib is the POSIX portability layer we're trying to eliminate.

glib is the higher level library we're trying to adopt.

The new approach will be more or less identical just different
naming convention.

I'll get my current glib series in a working state again and
push it somewhere public. That would enable you to base your
work on top of the glib series, using the new glib macros,
so that you're not blocked.

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list