[PATCH 2/2] xfixes: Use list.h macros in tracking hide count

Dan Nicholson dbn.lists at gmail.com
Mon Jun 14 11:38:10 PDT 2010


2010/6/14 Kristian Høgsberg <krh at bitplanet.net>:
> On Mon, Jun 14, 2010 at 10:56 AM, Tiago Vignatti
> <tiago.vignatti at nokia.com> wrote:
>> On Mon, Jun 14, 2010 at 03:25:23PM +0200, ext Kristian H�gsberg wrote:
>>>
>>> This is not a patch to be applied.
>>
>> why not?! :)
>
> Oh, I mentioned this in my first attempt to send out these patches
> (which git send-mail threw away when it couldn't find an smtp server).
>  I don't think it makes sense to go through the server and replace
> existing linked list code.  That code is already written and tested,
> and replacing it can only break stuff.

FWIW, the code that caused this issue was new. I was introducing a new
list and figured "oh, we have a nice list API now, I should probably
use it". I don't think anyone has proposed re-implementing any
existing lists with the new list API.

--
Dan


More information about the xorg-devel mailing list