diff options
author | Kenneth Graunke <[email protected]> | 2012-03-13 13:05:16 -0700 |
---|---|---|
committer | Kenneth Graunke <[email protected]> | 2012-03-26 10:21:57 -0700 |
commit | 7a348b91ce983c7efb1db61f36083f7d8d088f26 (patch) | |
tree | 8964d923dc88fdcc3f3bd1b0e6e4e6ca169c874b /src/glsl/standalone_scaffolding.cpp | |
parent | dca19a771156685895892740f687cee7cf84a8c9 (diff) |
glsl: Make ir_dereference_variable ctor assert the variable exists.
This also seems like a bad idea. There were too many instances for me
to thoroughly scan the code as I did with the last two patches, but a
quick scan indicated that most callers newly allocate a variable,
dereference it, or NULL-check. In some cases, it wasn't clear that the
value would be non-NULL, but they didn't check for error_type either.
At any rate, not checking for this is a bug, and assertions will trigger
it earlier and more reliably than returning error_type.
Signed-off-by: Kenneth Graunke <[email protected]>
Reviewed-by: Eric Anholt <[email protected]>
Diffstat (limited to 'src/glsl/standalone_scaffolding.cpp')
0 files changed, 0 insertions, 0 deletions