[PATCH] dix: Improve documentation of the DIX private data functions.

Aaron Plattner aplattner at nvidia.com
Wed Apr 28 12:34:49 PDT 2010


On Wed, Apr 28, 2010 at 07:23:29AM -0700, Tiago Vignatti wrote:
> On Wed, Apr 28, 2010 at 03:21:17AM +0200, ext Aaron Plattner wrote:
> > The functions exported by the devPrivates code were poorly documented.  I tried
> > to spruce it up a little.
> > 
> > Signed-off-by: Aaron Plattner <aplattner at nvidia.com>
> > ---
> > A lot of people have been touching code that uses these recently, so I thought
> > it might be nice to document some of the nuances of the interface.
> 
> Thanks Aaron. This looks much better:
> 
> Reviewed-by: Tiago Vignatti <tiago.vignatti at nokia.com>

Thanks, Tiago.

> But probably we might want to wait a bit because Keith is reworking
> devPrivates allocation behaviour.

I hope he's not changing the semantics of these functions, because a lot of
code would have to be changed if he does.  Plus, if he does it might be
nice to see the change to the documentation in the diff so it's clear
exactly how they're changing.


More information about the xorg-devel mailing list