On Mon, Nov 9, 2009 at 12:14 AM, Maarten Maathuis <[email protected]> wrote:
>> This is just an observation, but I do find the fact we have two things
>>
>> exaPixmapIsOffscreen(pPixmap)
>> and
>> !pExaPixmap->offscreen
>>
>> Like I know they are different but really seriously can anyone know
>> the difference just by reading this?
>>
>> EXA has become quite complicated recently and very difficult to trace
>> through, this naming doesn't help.
>>
>> Dave.
>
> I understand your concern and i appreciate you bringing this up. I
> will get back to you as soon as i have some decent ideas.
>
> Maarten.
>

exaPixmapIsOffscreen() is related to having a driver/vram/whatever pixmap

pExaPixmap->offscreen says if the offscreen pixmap is the current
choice of rendering to

The naming came from classic exa. Some better names might be:

exaPixmapHasOffscreen() and pExaPixmap->using_offscreen/use_offscreen


What do people think?

Maarten.
_______________________________________________
xorg-devel mailing list
[email protected]
http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to