From: Rob Clark <[email protected]> It's no extra overhead to do a _self_link() and it eliminates a class of potential problems.
Signed-off-by: Rob Clark <[email protected]> Dared-by: Jason Ekstrand <[email protected]> --- src/compiler/glsl/list.h | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/compiler/glsl/list.h b/src/compiler/glsl/list.h index a1c4d82..77e1f67 100644 --- a/src/compiler/glsl/list.h +++ b/src/compiler/glsl/list.h @@ -165,19 +165,18 @@ exec_node_get_prev(struct exec_node *n) } static inline void -exec_node_remove(struct exec_node *n) +exec_node_self_link(struct exec_node *n) { - n->next->prev = n->prev; - n->prev->next = n->next; - n->next = NULL; - n->prev = NULL; + n->next = n; + n->prev = n; } static inline void -exec_node_self_link(struct exec_node *n) +exec_node_remove(struct exec_node *n) { - n->next = n; - n->prev = n; + n->next->prev = n->prev; + n->prev->next = n->next; + exec_node_self_link(n); } static inline void -- 2.5.5 _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
