<div dir="ltr">On 10 April 2013 14:33, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</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">
<div class="HOEnZb"><div class="h5">On 04/10/2013 01:32 PM, Kenneth Graunke wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 04/09/2013 09:01 PM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The comment above glsl_type::name claimed that it could sometimes be<br>
NULL.  This was wrong--it is never NULL.  Many error handling paths<br>
would segfault if it were.<br>
<br>
Fix the comment and add assertions to validate that it really is never<br>
NULL.<br>
---<br>
  src/glsl/glsl_types.cpp | 4 ++++<br>
  src/glsl/glsl_types.h   | 3 +--<br>
  2 files changed, 5 insertions(+), 2 deletions(-)<br>
</blockquote>
<br>
I could've sworn I've actually seen it be NULL for anonymous structures.<br>
  What do those use?  ""?<br>
</blockquote>
<br></div></div>
They get a name like "#anon_struct_0001".  See the ast_struct_specifier constructor in glsl_parser_extras.cpp.  It may be worth mentioning this either in the commit message or in the code somewhere.<br></blockquote>
<div><br></div><div>I'll mention it in the commit message.<br></div></div></div></div>