struct dump_ctx is passed-by-pointer to nearly every function in vrend_shader.c as a monolithic data container, although none or only a few of its members are reassigned during each function call.
This patch refactors vrend_shader.c with a preference towards passing "const struct *dump_ctx" instead, and additionally passing affected members of dump_ctx by non-const pointer when reassignment of member values is required.
This makes it easier to understand the effects of each function call on the state of struct dump_ctx and should make it simpler to identify the causes of TGSI to GLSL conversion issues in the future.
Signed-off-by: Ryan Neph ryanneph@google.com
Issue #179 (closed)