Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Brian Paul
On 05/11/2012 04:26 PM, Jordan Justen wrote: On Fri, May 11, 2012 at 2:41 PM, Brian Paul wrote: On 05/11/2012 11:35 AM, Marek Olšák wrote: In that case, I think we should disable it in the state tracker if there's no driver support. If it's faster for svga, it's very likely faster for the othe

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Jordan Justen
On Fri, May 11, 2012 at 2:41 PM, Brian Paul wrote: > On 05/11/2012 11:35 AM, Marek Olšák wrote: >> In that case, I think we should disable it in the state tracker if >> there's no driver support. If it's faster for svga, it's very likely >> faster for the other DX9-level drivers too. > > Some View

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Brian Paul
On 05/11/2012 11:35 AM, Marek Olšák wrote: On Fri, May 11, 2012 at 7:17 PM, Brian Paul wrote: On 05/11/2012 11:08 AM, Marek Olšák wrote: I think a more preferable way to enable an extension for everybody by default is to do something like this: diff --git a/src/mesa/main/extensions.c b/src/

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Marek Olšák
On Fri, May 11, 2012 at 8:37 PM, Jordan Justen wrote: > On Fri, May 11, 2012 at 10:49 AM, Ian Romanick wrote: >> On 05/11/2012 10:35 AM, Marek Olšák wrote: >>> On Fri, May 11, 2012 at 7:17 PM, Brian Paul  wrote: So setting MESA_EXTENSION_OVERRIDE=-GL_NV_primitive_restart should be an op

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Jordan Justen
On Fri, May 11, 2012 at 10:49 AM, Ian Romanick wrote: > On 05/11/2012 10:35 AM, Marek Olšák wrote: >> On Fri, May 11, 2012 at 7:17 PM, Brian Paul  wrote: >>> So setting MESA_EXTENSION_OVERRIDE=-GL_NV_primitive_restart should be an >>> option.  The o(dummy_true) convention might prevent that, but I

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Ian Romanick
On 05/11/2012 10:35 AM, Marek Olšák wrote: On Fri, May 11, 2012 at 7:17 PM, Brian Paul wrote: On 05/11/2012 11:08 AM, Marek Olšák wrote: I think a more preferable way to enable an extension for everybody by default is to do something like this: diff --git a/src/mesa/main/extensions.c b/src/

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Marek Olšák
On Fri, May 11, 2012 at 7:17 PM, Brian Paul wrote: > On 05/11/2012 11:08 AM, Marek Olšák wrote: >> >> I think a more preferable way to enable an extension for everybody by >> default is to do something like this: >> >>> diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c >>> index

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Brian Paul
On 05/11/2012 11:08 AM, Marek Olšák wrote: I think a more preferable way to enable an extension for everybody by default is to do something like this: diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index a843a40..71b7721 100644 --- a/src/mesa/main/extensions.c +++ b/src/me

Re: [Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Marek Olšák
I think a more preferable way to enable an extension for everybody by default is to do something like this: > diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c > index a843a40..71b7721 100644 > --- a/src/mesa/main/extensions.c > +++ b/src/mesa/main/extensions.c > @@ -298,7 +298,

[Mesa-dev] [PATCH 5/8] main: enable NV_primitive_restart extension by default

2012-05-11 Thread Jordan Justen
Signed-off-by: Jordan Justen --- src/mesa/main/extensions.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c index a843a40..5389dfc 100644 --- a/src/mesa/main/extensions.c +++ b/src/mesa/main/extensions.c @@ -370,6 +370,8 @@ static c