LGTM，thanks. Reviewed-by: Zhigang Gong <[email protected]>
> -----Original Message----- > From: [email protected] [mailto:[email protected]] > On Behalf Of Eric Anholt > Sent: Saturday, January 25, 2014 3:01 AM > To: [email protected] > Subject: [PATCH] glamor: Disable definitions of GL extension prototypes to > avoid warnings. > > We're not using the extension prototypes, since you have to dlsym them > anyway. Disabling their definitions prevents them from being defined twice > (once by gl.h, once by glext.h). > > Signed-off-by: Eric Anholt <[email protected]> > --- > > I was about to send the pull req for the glamor reformat and initial build work, > but realized I was running with some default warnings disabled. This fixes > those. > > (It also would have gone away with the epoxy changes that come later). > > glamor/glamor_priv.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/glamor/glamor_priv.h b/glamor/glamor_priv.h index > 9f0c558..8ccf4fa 100644 > --- a/glamor/glamor_priv.h > +++ b/glamor/glamor_priv.h > @@ -35,8 +35,6 @@ > #endif > #include "glamor.h" > > -#define GL_GLEXT_PROTOTYPES > - > #ifdef GLAMOR_GLES2 > #include <GLES2/gl2.h> > #include <GLES2/gl2ext.h> > -- > 1.8.5.3 > > _______________________________________________ > [email protected]: X.Org development > Archives: http://lists.x.org/archives/xorg-devel > Info: http://lists.x.org/mailman/listinfo/xorg-devel _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel
