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

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

 



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@xxxxxxxxx>
> > > > ---
> > > Reviewed-by: Erik Skultety <eskultet@xxxxxxxxxx>
> > > 
> > > 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 :|

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