[PATCH] kms: Fix warning XNFprintf is deprecated (v2)

2011-03-01 Thread Michel Dänzer
[Dropping dri-devel from CC again, X driver patches aren't really on topic there] On Sam, 2011-02-26 at 21:04 +0100, Sedat Dilek wrote: > The first take of my patch missed the breakage against pre-1.10 > xservers and now I know XNFasprintf() is not a 1:1 replacement > for XNFprintf(). What would

Re: [PATCH] kms: Fix warning XNFprintf is deprecated (v2)

2011-03-01 Thread Michel Dänzer
[Dropping dri-devel from CC again, X driver patches aren't really on topic there] On Sam, 2011-02-26 at 21:04 +0100, Sedat Dilek wrote: > The first take of my patch missed the breakage against pre-1.10 > xservers and now I know XNFasprintf() is not a 1:1 replacement > for XNFprintf(). What would

Re: [PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-27 Thread Michel Dänzer
[Dropping dri-devel from CC] On Fre, 2011-02-25 at 22:07 +0100, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at > /usr/include/xorg/os.h:273) > > Signed-off-by:

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-26 Thread Sedat Dilek
On Sat, Feb 26, 2011 at 6:08 PM, Alan Coopersmith 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 >>

[PATCH] kms: Fix warning XNFprintf is deprecated (v2)

2011-02-26 Thread Sedat Dilek
The first take of my patch missed the breakage against pre-1.10 xservers and now I know XNFasprintf() is not a 1:1 replacement for XNFprintf(). The idea for the fix seemed to be so obvious, but as usual "really" test the patch you are sending to any ML. I am awfully sorry for the overhasty bad hack

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-26 Thread Michel Dänzer
[Dropping dri-devel from CC] On Fre, 2011-02-25 at 22:07 +0100, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at > /usr/include/xorg/os.h:273) > > Signed-off-by:

Re: [PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-26 Thread Sedat Dilek
On Sat, Feb 26, 2011 at 6:08 PM, Alan Coopersmith 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 >>

[PATCH] kms: Fix warning XNFprintf is deprecated (v2)

2011-02-26 Thread Sedat Dilek
The first take of my patch missed the breakage against pre-1.10 xservers and now I know XNFasprintf() is not a 1:1 replacement for XNFprintf(). The idea for the fix seemed to be so obvious, but as usual "really" test the patch you are sending to any ML. I am awfully sorry for the overhasty bad hack

Re: [PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-26 Thread Alan Coopersmith
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.

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-26 Thread Alan Coopersmith
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.

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Sedat Dilek
Noticed when building ati/radeon ddx against xserver 1.10-rc3. [ build.log ] ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at /usr/include/xorg/os.h:273) Signed-off-by: Sedat Dilek --- src/radeon_kms.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) di

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Alex Deucher
Pushed a variant of this without breaking pre-1.10 xservers. cb4dc7ab66016d0c980800f8dbf71bbe993889d3 On Fri, Feb 25, 2011 at 4:07 PM, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is dep

Re: [PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Alex Deucher
Pushed a variant of this without breaking pre-1.10 xservers. cb4dc7ab66016d0c980800f8dbf71bbe993889d3 On Fri, Feb 25, 2011 at 4:07 PM, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is depr

Re: [PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Alan Coopersmith
On 02/25/11 01:07 PM, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at > /usr/include/xorg/os.h:273) > > Signed-off-by: Sedat Dilek > --- > src/radeon_kms.c |

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Alan Coopersmith
On 02/25/11 01:07 PM, Sedat Dilek wrote: > Noticed when building ati/radeon ddx against xserver 1.10-rc3. > > [ build.log ] > ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at > /usr/include/xorg/os.h:273) > > Signed-off-by: Sedat Dilek > --- > src/radeon_kms.c |

[PATCH] kms: Fix warning XNFprintf is deprecated

2011-02-25 Thread Sedat Dilek
Noticed when building ati/radeon ddx against xserver 1.10-rc3. [ build.log ] ../../src/radeon_kms.c:412:5: warning: 'XNFprintf' is deprecated (declared at /usr/include/xorg/os.h:273) Signed-off-by: Sedat Dilek --- src/radeon_kms.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) di