On Sat, 2016-04-02 at 17:09 -0400, Rob Clark wrote:
> From: Rob Clark <robcl...@freedesktop.org>
>
> It's no extra overhead to do a _self_link() and it eliminates a class of
> potential problems.
it can also hide actual programming mistakes that would otherwise be
immediately visible... does this actually help something specific?
Iago
> Signed-off-by: Rob Clark <robcl...@freedesktop.org>
> Dared-by: Jason Ekstrand <ja...@jlekstrand.net>
> ---
> 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
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
