On Fri, 23 Dec 2011 15:18:21 -0800 "Ian Romanick" <i...@freedesktop.org> wrote:
> From: Ian Romanick <ian.d.roman...@intel.com> > > Signed-off-by: Ian Romanick <ian.d.roman...@intel.com> > --- > 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 <jbar...@virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel