Re: [PATCH v2 4/9] rbtree: place easiest case first in rb_erase()

2012-08-05 Thread Rik van Riel
On 08/02/2012 06:34 PM, Michel Lespinasse wrote: In rb_erase, move the easy case (node to erase has no more than 1 child) first. I feel the code reads easier that way. Signed-off-by: Michel Lespinasse Reviewed-by: Rik van Riel -- All rights reversed -- To unsubscribe from this list: send

Re: [PATCH v2 4/9] rbtree: place easiest case first in rb_erase()

2012-08-05 Thread Rik van Riel
On 08/02/2012 06:34 PM, Michel Lespinasse wrote: In rb_erase, move the easy case (node to erase has no more than 1 child) first. I feel the code reads easier that way. Signed-off-by: Michel Lespinassewal...@google.com Reviewed-by: Rik van Riel r...@redhat.com -- All rights reversed -- To

[PATCH v2 4/9] rbtree: place easiest case first in rb_erase()

2012-08-02 Thread Michel Lespinasse
In rb_erase, move the easy case (node to erase has no more than 1 child) first. I feel the code reads easier that way. Signed-off-by: Michel Lespinasse --- lib/rbtree.c | 35 ++- 1 files changed, 18 insertions(+), 17 deletions(-) diff --git a/lib/rbtree.c

[PATCH v2 4/9] rbtree: place easiest case first in rb_erase()

2012-08-02 Thread Michel Lespinasse
In rb_erase, move the easy case (node to erase has no more than 1 child) first. I feel the code reads easier that way. Signed-off-by: Michel Lespinasse wal...@google.com --- lib/rbtree.c | 35 ++- 1 files changed, 18 insertions(+), 17 deletions(-) diff --git