Re: [Mesa-dev] [PATCH 3/4] mesa: remove ctx->Const.sRGBCapable

2012-01-25 Thread Dave Airlie
On Wed, Jan 25, 2012 at 6:58 AM, Dave Airlie wrote: > On Wed, Jan 25, 2012 at 1:53 AM, Ian Romanick wrote: >> On 01/24/2012 04:39 PM, Marek Olšák wrote: >>> >>> It always had the same value as ctx->Extensions.EXT_framebuffer_sRGB. >> >> >> I'd be curious why Dave added this flag in the first plac

Re: [Mesa-dev] [PATCH 3/4] mesa: remove ctx->Const.sRGBCapable

2012-01-24 Thread Dave Airlie
On Wed, Jan 25, 2012 at 1:53 AM, Ian Romanick wrote: > On 01/24/2012 04:39 PM, Marek Olšák wrote: >> >> It always had the same value as ctx->Extensions.EXT_framebuffer_sRGB. > > > I'd be curious why Dave added this flag in the first place.  It's... odd. >  In any case, So would I, but I think I w

Re: [Mesa-dev] [PATCH 3/4] mesa: remove ctx->Const.sRGBCapable

2012-01-24 Thread Ian Romanick
On 01/24/2012 04:39 PM, Marek Olšák wrote: It always had the same value as ctx->Extensions.EXT_framebuffer_sRGB. I'd be curious why Dave added this flag in the first place. It's... odd. In any case, Reviewed-by: Ian Romanick --- src/mesa/drivers/dri/intel/intel_context.c |1 - sr

[Mesa-dev] [PATCH 3/4] mesa: remove ctx->Const.sRGBCapable

2012-01-24 Thread Marek Olšák
It always had the same value as ctx->Extensions.EXT_framebuffer_sRGB. --- src/mesa/drivers/dri/intel/intel_context.c |1 - src/mesa/main/fbobject.c |2 +- src/mesa/main/framebuffer.c|2 +- src/mesa/main/mtypes.h |3 --- src/mesa