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

[Maarten Maathuis]

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)

Maarten.