> -----Original Message----- > From: Anthony PERARD [mailto:anthony.per...@citrix.com] > Sent: 05 December 2018 13:59 > To: Paul Durrant <paul.durr...@citrix.com> > Cc: Kevin Wolf <kw...@redhat.com>; Stefano Stabellini > <sstabell...@kernel.org>; qemu-bl...@nongnu.org; qemu-devel@nongnu.org; > Max Reitz <mre...@redhat.com>; xen-de...@lists.xenproject.org > Subject: Re: [PATCH 04/18] xen: create xenstore areas for XenDevice-s > > On Wed, Dec 05, 2018 at 12:43:57PM +0000, Paul Durrant wrote: > > > > > + value = g_strdup_vprintf(fmt, ap); > > > > > > > > Looks like g_vasprintf() would be better, since it returns the > lenght as > > > > well. > > > > > > > > > > Yes. > > > > I tried this and it appears not to exist in the version of glib in my > environment so I guess I'll stick with g_strdup_printf(). > > It's probably because you need to include "glib/gprintf.h", I've > suggested it because I've seen the function use elsewhere in QEMU. But > g_strdup_printf is fine too. > > https://developer.gnome.org/glib/stable/glib-String-Utility- > Functions.html#g-vasprintf
Ah, that's what I needed. Thanks, Paul > > -- > Anthony PERARD
