program: fix out of bounds array accesses and other bad things
authornobled <nobled@dreamwidth.org>
Thu, 8 Dec 2011 21:07:18 +0000 (21:07 +0000)
committernobled <nobled@dreamwidth.org>
Sun, 18 Dec 2011 17:12:25 +0000 (17:12 +0000)
commitf71984ee5b4a7369ce3cc86fb0253ac66d80388c
tree45f3bef297f4bb18eac312708d3002ca58dbbfa3
parent03bfdb2bad3e4d7c3c53bdfa8a392979ccf89baa
program: fix out of bounds array accesses and other bad things

Noticed a "warning: array subscript is above array bounds" given at one of
the existing sanity-check asserts. Turns out all the arrays of strings
haven't matched the corresponding enum values in a while, if ever.

I didn't know the proper names for any of these and couldn't find
them in the base specs aside from "result.pointsize" in
ARB_vertex_program, so I just filled in the enum's value
as was done with other slots.

Also add four STATIC_ASSERT()s to be sure and catch future additions
or bumps to MAX_VARYING/etc again, and some more non-static asserts
where there weren't any before.

(Note, the fragment enum that corresponded to result.color(half) was removed in
8d475822e6e19fa79719c856a2db5b6a205db1b9.)

Reviewed-by: Brian Paul <brianp@vmware.com>
src/mesa/program/prog_print.c