On Tue, Dec 16, 2014 at 1:05 AM, Jason Ekstrand <ja...@jlekstrand.net> wrote:
> Some time while refactoring things to make it look nicer before pushing to
> master, I completely broke the function. This fixes it to be correct.
> Just goes to show you why you souldn't push code that has no users yet...
>
> Signed-off-by: Jason Ekstrand <jason.ekstr...@intel.com>
> ---
> src/glsl/list.h | 4 +---
> 1 file changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/src/glsl/list.h b/src/glsl/list.h
> index 924c729..4774ccc 100644
> --- a/src/glsl/list.h
> +++ b/src/glsl/list.h
> @@ -534,9 +534,7 @@ exec_list_validate(const struct exec_list *list)
> * either require C++ or assume the exec_node is embedded in a structure
> * which is not the case for this function.
> */
> - for (node = exec_list_get_head_const(list);
> - !exec_node_is_tail_sentinel(node);
> - node = exec_node_get_next_const(node)) {
> + for (node = list->head; node->next != NULL; node = node->next) {
> assert(node->next->prev == node);
> assert(node->prev->next == node);
> }
> --
> 2.2.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Just curious before I put a r-b on this, what exactly was broken about it?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
