[Mesa-dev] [PATCH] intel: Change vendor string to "Intel® Open Source Technology Center".

[Oliver McFadden]

On Fri, Jun 01, 2012 at 09:18:44AM +0300, Pekka Paalanen wrote:
> On Thu, 31 May 2012 16:19:17 -0700
> Kenneth Graunke <kenneth at whitecape.org> wrote:
> 
> > Tungsten Graphics has not existed for several years, and the majority of
> > ongoing development and support is done by Intel.  I chose to include
> > "Open Source Technology Center" to distinguish it from, say, the closed
> > source Windows OpenGL driver.
> > 
> > The one downside to this patch is that applications that pattern match
> > against "Intel" may start applying workarounds meant for the Windows
> > driver.  However, it does seem like the right thing to do.
> > 
> > This does change oglconform behavior.
> > 
> > Acked-by: Eric Anholt <eric at anholt.net>
> > Acked-by: Ian Romanick <ian.d.romanick at intel.com>
> > Cc: Eugeni Dodonov <eugeni at dodonov.net>
> > Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> > ---
> >  src/mesa/drivers/dri/intel/intel_context.c |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c
> > index 9deb4ca..712acb8 100644
> > --- a/src/mesa/drivers/dri/intel/intel_context.c
> > +++ b/src/mesa/drivers/dri/intel/intel_context.c
> > @@ -72,7 +72,7 @@ intelGetString(struct gl_context * ctx, GLenum name)
> >  
> >     switch (name) {
> >     case GL_VENDOR:
> > -      return (GLubyte *) "Tungsten Graphics, Inc";
> > +      return (GLubyte *) "Intel® Open Source Technology Center";
> 
> Hi,
> 
> that is an utf-8 character there, right?
> Is that really appropriate, given the API and all?
> 
> I don't know about character encoding stuff, but that looks surprising.
> Just wondering...

Furthermore it actually doesn't render correctly with my encodings
(en_GB and en_US: UTF-8 and ISO-8859-1, and various Finnish and Swedish
ones left out for brevity.

Okay, problem on my end, but I'm probably not the only one (or maybe I
am just an idiot who can't setup encodings correctly.)

Wouldn't "Intel(R) Open Source ..." be sufficient?


> 
> 
> Thanks,
> pq
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev

-- 
Oliver McFadden.