[PATCH] [pixmap] rename/convert devKind and devPrivate fields

Maarten Maathuis madman2003 at gmail.com
Thu Oct 9 14:20:11 PDT 2008


On Thu, Oct 9, 2008 at 11:08 PM, Dave Airlie <airlied at gmail.com> wrote:
> On Fri, Oct 10, 2008 at 3:18 AM, Maarten Maathuis <madman2003 at gmail.com> wrote:
>> On Thu, Oct 9, 2008 at 9:34 AM, Michel Dänzer
>> <michel at tungstengraphics.com> wrote:
>>> On Wed, 2008-10-08 at 19:20 +0200, Maarten Maathuis wrote:
>>>> On Tue, Oct 7, 2008 at 8:04 PM, Maarten Maathuis <madman2003 at gmail.com> wrote:
>>>> > These names are supposedly historical, but everyone seems to use them
>>>> > for the same purpose.
>>>> > I am aware this may require a few minor changes to (for example) exa
>>>> > drivers with driver based memory allocations.
>>>> > I personally believe in having clear names rather than some obscure
>>>> > remnant of the past. I removed the devPrivate considering the void
>>>> > pointer is the only useful part anyway.
>>>> >
>>>> > It would be nice to know if all !xfree86 has the equivilant of
>>>> > VIDEODRV_ABI_VERSION.
>>>> >
>>>> > Anyway, share your thoughts, comments and such.
>>>> >
>>>> > Maarten.
>>>> >
>>>> > PS. i can commit these myself, this is just for review, considering
>>>> > it's a non-trivial change.
>>>> >
>>>>
>>>> Any thoughts about other stuff needing change in the pixmap (or maybe
>>>> the closely related drawable)? I'm openminded to change, so it seems
>>>> sensible to check if there are any other problems out there. Without
>>>> any further complaints i'll probably go ahead with this in a few days.
>>>
>>> I'm not sure this is worth the pain for drivers...
>>>
>>>
>>> --
>>> Earthling Michel Dänzer           |          http://tungstengraphics.com
>>> Libre software enthusiast         |          Debian, X and DRI developer
>>>
>>>
>>
>> Are you thinking of anything besides ddx drivers? (i checked ati, it
>> has about 5 instances, intel about 12, a simple macro could serve as a
>> drop in replacement)
>
> It totally messes up the ability to do git bisections and such for 0 value other
> than changing some names. The gain isn't outweighing much.
>
> Dave.
>
>>
>> Maarten.
>> _______________________________________________
>> xorg mailing list
>> xorg at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/xorg
>>
>

The first compelling argument, i guess i'll stick to adding a comment
here and there.

Maarten.



More information about the xorg mailing list