[Mesa-dev] [PATCH 059/133] glsl/list: Fix the exec_list_validate function

Jason Ekstrand jason at jlekstrand.net
Mon Dec 15 22:05:09 PST 2014


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.ekstrand at 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



More information about the mesa-dev mailing list