[PATCH] kms: Fix warning XNFprintf is deprecated

Sedat Dilek sedat.dilek at googlemail.com
Sat Feb 26 12:05:18 PST 2011


On Sat, Feb 26, 2011 at 6:08 PM, Alan Coopersmith
<alan.coopersmith at oracle.com> wrote:
> On 02/25/11 02:32 PM, Alan Coopersmith wrote:
>>> -    busid = XNFprintf("pci:%04x:%02x:%02x.%d",
>>> +    busid = XNFasprintf("pci:%04x:%02x:%02x.%d",
>>
>> A straight replacement like that, with no #ifdef's will break people building
>> for Xorg 1.9 and older, since XNFasprintf is new in Xorg 1.10.
>
> Oops, I failed to notice the other problem with this patch, which was
> reported in https://bugs.freedesktop.org/show_bug.cgi?id=34756 - the two
> API's are different - the replacement should be:
>
>        XNFasprintf(&busid, "pci:%04x:%02x:%02x.%d",
>                    dev->domain, dev->bus, dev->dev, dev->func);
>
> since it follows the style of the libc asprintf() function.
>
> The compiler should have yelled loudly about the original patch when you
> tried building it, as Michel noted in his revert message.
>
> --
>        -Alan Coopersmith-        alan.coopersmith at oracle.com
>         Oracle Solaris Platform Engineering: X Window System
>
>

[ CC kdebugs at hotmail.com (Sorry, could not decrypt your real name) ]

A short hello from my bday party and sorry for the breakage.

I have gone through all replies and I have built and installed a new
ddx against xserver-1.10-rc3 with a v2 of my initial patch (which I
will send separately).

- Sedat -


More information about the xorg-driver-ati mailing list