Re: [PATCH 5/8] util: use gnu_printf format attribute

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



Hi

On Wed, Oct 9, 2019 at 8:03 AM David Gibson <david@xxxxxxxxxxxxxxxxxxxxx> wrote:
>
> On Mon, Oct 07, 2019 at 11:18:03PM +0400, marcandre.lureau@xxxxxxxxxx wrote:
> > From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> >
> > Fix mingw cross-compilation: the use of "%z" format and other requires
> > mingw ansi stdio.
>
> Again, can you connec the dots a bit more as to why this breaks mingw.

https://sourceforge.net/p/mingw-w64/wiki2/printf%20and%20scanf%20family/
https://sourceforge.net/p/mingw-w64/wiki2/gnu%20printf/

dtc uses non-portable formats. Using gnu_printf attributes (for
warnings) in combination with __USE_MINGW_ANSI_STDIO allows to build
for win32.

>
> >
> > Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> > ---
> >  util.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/util.h b/util.h
> > index ca5cb52..347b305 100644
> > --- a/util.h
> > +++ b/util.h
> > @@ -12,7 +12,7 @@
> >   */
> >
> >  #ifdef __GNUC__
> > -#define PRINTF(i, j) __attribute__((format (printf, i, j)))
> > +#define PRINTF(i, j) __attribute__((format (gnu_printf, i, j)))
> >  #define NORETURN     __attribute__((noreturn))
> >  #else
> >  #define PRINTF(i, j)
>
> --
> David Gibson                    | I'll have my music baroque, and my code
> david AT gibson.dropbear.id.au  | minimalist, thank you.  NOT _the_ _other_
>                                 | _way_ _around_!
> http://www.ozlabs.org/~dgibson





[Index of Archives]     [Device Tree]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux