<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 16, 2014 at 2:49 PM, Connor Abbott <span dir="ltr"><<a href="mailto:cwabbott0@gmail.com" target="_blank">cwabbott0@gmail.com</a>></span> wrote:<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 Tue, Dec 16, 2014 at 1:05 AM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Some time while refactoring things to make it look nicer before pushing to<br>
> master, I completely broke the function.  This fixes it to be correct.<br>
> Just goes to show you why you souldn't push code that has no users yet...<br>
><br>
> Signed-off-by: Jason Ekstrand <<a href="mailto:jason.ekstrand@intel.com">jason.ekstrand@intel.com</a>><br>
> ---<br>
>  src/glsl/list.h | 4 +---<br>
>  1 file changed, 1 insertion(+), 3 deletions(-)<br>
><br>
> diff --git a/src/glsl/list.h b/src/glsl/list.h<br>
> index 924c729..4774ccc 100644<br>
> --- a/src/glsl/list.h<br>
> +++ b/src/glsl/list.h<br>
> @@ -534,9 +534,7 @@ exec_list_validate(const struct exec_list *list)<br>
>      * either require C++ or assume the exec_node is embedded in a structure<br>
>      * which is not the case for this function.<br>
>      */<br>
> -   for (node = exec_list_get_head_const(list);<br>
> -        !exec_node_is_tail_sentinel(node);<br>
> -        node = exec_node_get_next_const(node)) {<br>
> +   for (node = list->head; node->next != NULL; node = node->next) {<br>
>        assert(node->next->prev == node);<br>
>        assert(node->prev->next == node);<br>
>     }<br>
> --<br>
> 2.2.0<br>
><br>
</div></div>> _______________________________________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
<br>
Just curious before I put a r-b on this, what exactly was broken about it?<br></blockquote><div><br></div><div>I should probably have said this in the commit message, but the list_get_head and get_next return null if it's empty or you're at the end.  Yes, I could have used them and tweaked it differently, but I figured consistency was probably better.<br></div><div>--Jason <br></div></div></div></div>