[PATCH xf86-video-trident] Fix up deprecated/obsolete code.

Alan Coopersmith alan.coopersmith at oracle.com
Tue Aug 3 09:52:47 PDT 2010


Vignatti Tiago (Nokia-MS/Helsinki) wrote:
> On Tue, Aug 03, 2010 at 05:11:44PM +0200, ext Trevor Woerner wrote:
>> Hi Tiago,
>>
>> Thanks for having a look at my patch. I was surprised nobody picked up
>> on the idea of fixing up these xmalloc()/xfree()/etc. functions
>> already, but then I found this in the archives from Alan Coopersmith:
>>
>> http://lists.x.org/archives/xorg-devel/2010-July/010804.html
>>
>> Does this mean we shouldn't be fixing these things?
> 
> no, the opposite! This kind of fix is very welcome. I've done myself a similar
> patch with nv driver. I remember also Peter done something with evdev and so on.

Note the important distinction between client libraries we can't change
(libX*) and the X server and drivers that are changed.   My message you
point to is about the client library API, not the drivers - as Tiago
correctly points out, we want these removed from the drivers, since the
server API has deprecated them.

Both sets of API's had confusingly similarly named malloc wrapping functions.
One must stay, the other is going away.

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list