[Mesa-dev] [PATCH 2/4] intel: Drop the version override code now that we don't have any left.

Eric Anholt eric at anholt.net
Thu Jan 12 16:48:18 PST 2012


On Thu, 12 Jan 2012 16:21:59 -0800, Kenneth Graunke <kenneth at whitecape.org> wrote:
> On 01/12/2012 04:08 PM, Eric Anholt wrote:
> > Fixes a compiler warning.
> > ---
> >   src/mesa/drivers/dri/intel/intel_extensions.c |    8 --------
> >   1 files changed, 0 insertions(+), 8 deletions(-)
> >
> > diff --git a/src/mesa/drivers/dri/intel/intel_extensions.c b/src/mesa/drivers/dri/intel/intel_extensions.c
> > index 7aa040c..0ce452f 100644
> > --- a/src/mesa/drivers/dri/intel/intel_extensions.c
> > +++ b/src/mesa/drivers/dri/intel/intel_extensions.c
> > @@ -41,14 +41,6 @@ void
> >   intelInitExtensions(struct gl_context *ctx)
> >   {
> >      struct intel_context *intel = intel_context(ctx);
> > -   char *override = getenv("MESA_GL_VERSION_OVERRIDE");
> > -   int override_major, override_minor;
> > -   int override_version = 0;
> > -
> > -   if (override&&
> > -       sscanf(override, "%u.%u",&override_major,&override_minor) == 2) {
> > -      override_version = override_major * 10 + override_minor;
> > -   }
> >
> >      ctx->Extensions.ARB_draw_elements_base_vertex = true;
> >      ctx->Extensions.ARB_explicit_attrib_location = true;
> 
> I would rather keep this...it seems like it will continue to be useful 
> as we move on to implementing 3.1, 3.2, 4.0, and so on.  There's not 
> really a lot gained by removing it, either...

git log intel_extensions will find it quickly, and otherwise I'll be
seeing this error every compile for the next 3 months.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20120112/dead57c3/attachment.pgp>


More information about the mesa-dev mailing list