<div dir="ltr">On 26 November 2013 00:02, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Add predicates to query if a GLSL type is or contains an image.<br>
Rename sampler_coordinate_components() to coordinate_components() and<br>
fix it to support calculating the number of coordinate dimensions of<br>
image types as well as sampler types.<br>
---<br>
 src/glsl/builtin_functions.cpp        |  2 +-<br>
 src/glsl/glsl_types.cpp               | 95 +++++++++++++++++++++++++----------<br>
 src/glsl/glsl_types.h                 | 19 ++++++-<br>
 src/glsl/tests/sampler_types_test.cpp |  2 +-<br>
 4 files changed, 88 insertions(+), 30 deletions(-)<br></blockquote><div><br></div><div>If the calls to unreachable() are replaced with assertions (as we've discussed elsewhere on the list), this patch is:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>><br>

<br>I don't have a strong opinion about Topi's suggestion to separate the sampler and image cases--I could go either way.<br></div></div></div></div>