On Tue, Sep 30, 2014 at 8:50 AM, Ian Romanick <i...@freedesktop.org> wrote:
> On 09/20/2014 07:41 PM, Matt Turner wrote:
>> On Sat, Sep 20, 2014 at 6:40 PM, Chris Forbes <chr...@ijw.co.nz> wrote:
>>> diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c
>>> index 79d2e94..c11ad4f 100644
>>> --- a/src/mesa/main/shaderapi.c
>>> +++ b/src/mesa/main/shaderapi.c
>>> @@ -105,6 +105,7 @@ _mesa_get_shader_flags(void)
>>> void
>>> _mesa_init_shader_state(struct gl_context *ctx)
>>> {
>>> + int i;
>>
>> In context, this declaration looks odd. Move it below the two just
>> after this hunk?
>
> Not in core Mesa where we have to do dumb ol' C89. :(
Move it after the other two variable declarations...
/* Device drivers may override these to control what kind of instructions
* are generated by the GLSL compiler.
*/
struct gl_shader_compiler_options options;
gl_shader_stage sh;
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
