On Thu, Dec 04 2014, Ian Romanick wrote:
> Shouldn't type be glsl_base_type instead of unsigned?

Yes, thanks for the catch. Fixed.

>> +         if (shadow)
>> +            return (array ? sampler2DArrayShadow_type : sampler2DShadow_type
>> +);
>
> It's weird that the closing paren ended up here.  I don't think the
> parens are necessary, so maybe drop them?

This was just a stray newline, not wrapping or anything. Thanks for
catching it.

I agree that the parentheses aren't required, but they are idiomatic
with all of the surrounding code. So I'll leave them for now.

-Carl

Attachment: pgpDZKI2unkAL.pgp
Description: PGP signature

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to