glsl: encapsulate binding validation and setting
authorTimothy Arceri <timothy.arceri@collabora.com>
Sat, 14 Nov 2015 13:55:29 +0000 (00:55 +1100)
committerTimothy Arceri <t_arceri@yahoo.com.au>
Fri, 20 Nov 2015 20:27:30 +0000 (07:27 +1100)
commit64710db66461e5ccfaf7667971bc5e513b9ce547
tree628fb3d39d4e6fadd949189a24491d0947901f30
parentdb3c36aedfa2e92c2cf1c17a096c1b5e7cd51c42
glsl: encapsulate binding validation and setting

This change moves the binding layout handing code into an apply
function to be consistent with other helper functions in the ast
code, and to encapsulate the code so that when we introduce
compile time constants the code will be much cleaner.

One small downside is for unnamed interface blocks we will now
be revalidating the binding for each member its applied to.
However this seems a small sacrifice in order to have code which
is readable.

We also remove the incorrect comment in the named interface code
about propagating bindings to members which seems to have been
copied from the unnamed interface code.

Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
src/glsl/ast_to_hir.cpp