Re: [Mesa-dev] [PATCH 1/7] mesa: clean up #includes in ff_fragment_shader.cpp

2015-10-01 Thread Brian Paul
On 10/01/2015 01:03 AM, Tapani Pälli wrote: There seems to be some 'extra includes', not sure if you want to start going through them at this same time? Right, I'm not too concerned with that now. -Brian ___ mesa-dev mailing list mesa-dev@lists.free

Re: [Mesa-dev] [PATCH 1/7] mesa: clean up #includes in ff_fragment_shader.cpp

2015-10-01 Thread Tapani Pälli
There seems to be some 'extra includes', not sure if you want to start going through them at this same time? As example sampler.cpp does not need to include glsl/ir_visitor.h since it comes with glsl/ir.h. Anyway nice cleanup, series is Reviewed-by: Tapani Pälli On 10/01/2015 12:35 AM, Brian

[Mesa-dev] [PATCH 1/7] mesa: clean up #includes in ff_fragment_shader.cpp

2015-09-30 Thread Brian Paul
Get rid of "../glsl/" paths. Sort alphabetically. --- src/mesa/main/ff_fragment_shader.cpp | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/src/mesa/main/ff_fragment_shader.cpp b/src/mesa/main/ff_fragment_shader.cpp index c682892..c0030bc 100644 --- a/