[PATCH] Use PRId32/PRIx32 from inttypes.h for printf of CARD32 values.
Thomas Dickey
dickey at his.com
Sun Oct 4 03:57:44 PDT 2009
On Sun, 4 Oct 2009, Mark Kettenis wrote:
>>> @@ -219,7 +219,7 @@ port_rep_inb(xf86Int10InfoPtr pInt,
>>> register int inc =3D d_f ? -1 : 1;
>>> CARD32 dst =3D base;
>>> if (PRINT_PORT && DEBUG_IO_TRACE())
>>> - ErrorF(" rep_insb(%#x) %ld bytes at %8.8lx %s\n",
>>> + ErrorF(" rep_insb(%#x) %" PRId32 " bytes at %8.8" PRIx32 " %s\n",
>>> port, count, base, d_f ? "up" : "down");
>>> while (count--) {
>>> MEM_WB(pInt, dst, x_inb(port));
>>
>> Is this portable beyond glibc? Alan? Matthieu?
>
> The PRId32 and PRIx32 defines are part of C99.
>
> That said, the consensus among many groups of developers is that they
> are ugly, make the code harder to read and can be easily avoided by
> using appropriate casts.
beyond being merely ugly, different versions of gcc give warnings for
combinations that are quiet in other versions of gcc.
* something to consider for developers who do port code...
--
Thomas E. Dickey
http://invisible-island.net
ftp://invisible-island.net
More information about the xorg-devel
mailing list