On 02/10/2011 07:38 AM, Matthias Bolte wrote:
> I get errors like this related to %lld in format strings:
>
> virsh.c: In function 'cmdDomblkstat':
> virsh.c:990:9: warning: unknown conversion type character 'l' in
> format [-Wformat]
>
> The problem goes away when I replace the define for vshPrint
>
> #define vshPrint(ctl, ...) fprintf(stdout, __VA_ARGS__)
>
> by this function
>
> static void
> vshPrint(vshControl *ctl ATTRIBUTE_UNUSED, const char *format, ...)
> {
> va_list ap;
>
> va_start(ap, format);
> vfprintf(stdout, format, ap);
> va_end(ap);
> }
Well yeah, writing a wrapper functions makes the warnings go away if the
wrapper doesn't have the write __attribute__ attached to it; but it
doesn't fix the real problem, which is that we aren't using the
fprintf-posix gnulib module, and therefore, we will still cause issues
on mingw trying to format with %lld.
A better alternative is to instead use [v]asprintf (where we ARE using
the gnulib module, and %lld works), and print that resulting string.
I'll propose a patch.
>
> I'm not sure why this became a problem now, as vshPrint is a define
> since 2006 and virsh used to compile before. Maybe this is an issue
> with gnulib in the current libvirt-0.8.8-rc1 tarball, as I'm testing
> based on this tarball it.
I'm not sure either, unless you have recently upgraded mingw headers and
the headers recently added a gcc attribute to fprintf where it used to
not have one.
--
Eric Blake eblake@xxxxxxxxxx +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list