On Fri, 23 Dec 2011 15:18:21 -0800
"Ian Romanick" <[email protected]> wrote:

> From: Ian Romanick <[email protected]>
> 
> Signed-off-by: Ian Romanick <[email protected]>
> ---
>  glx/glxcmds.c      |   13 ++++++++++---
>  glx/glxdri.c       |   10 +++++++++-
>  glx/glxdri2.c      |    5 ++++-
>  glx/glxdriswrast.c |    4 +++-
>  glx/glxscreens.h   |    5 ++++-
>  5 files changed, 30 insertions(+), 7 deletions(-)

You might include some verbiage in the changelog about why you're
adding attribs here (to support the later stuff).  Other than that it
looks like a straightforward addition.

Reviewed-by: Jesse Barnes <[email protected]>

-- 
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to