]> www.infradead.org Git - users/mchehab/rasdaemon.git/commitdiff
rbtree.h: Fix an issue introduced by checkpatch logic
authorMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 16 Jul 2024 08:12:42 +0000 (10:12 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 16 Jul 2024 08:12:42 +0000 (10:12 +0200)
Checkpatch actually broke RB_EMPTY_ROOT macro. It was defined
as:

#define RB_EMPTY_ROOT(root)    ((root)->rb_node == NULL)

It ended replacing it by:
((root)->!rb_node)

Which is not the way we espect it. Weird enough, this was compiling.
Anyway, what we want, instead, is:

#define RB_EMPTY_ROOT(root)    (!(root)->rb_node)

Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
rbtree.h

index e8c38de04d1d9ab11edc10f7e1a74dbdece4addf..555b3dca7c97463582d61964c61fcd05fae7c7ca 100644 (file)
--- a/rbtree.h
+++ b/rbtree.h
@@ -134,7 +134,7 @@ static inline void rb_set_color(struct rb_node *rb, int color)
 #define RB_ROOT        (struct rb_root) { NULL, }
 #define        rb_entry(ptr, type, member) container_of(ptr, type, member)
 
-#define RB_EMPTY_ROOT(root)    ((root)->!rb_node)
+#define RB_EMPTY_ROOT(root)    (!(root)->rb_node)
 #define RB_EMPTY_NODE(node)    (rb_parent(node) == node)
 #define RB_CLEAR_NODE(node)    (rb_set_parent(node, node))