git: 52d4f5b16a15 - stable/13 - rb_tree: Use void casts for RB_AUGMENT_CHECK with unused return value.
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Wed, 12 Oct 2022 03:00:28 UTC
The branch stable/13 has been updated by dougm: URL: https://cgit.FreeBSD.org/src/commit/?id=52d4f5b16a150c7d6227f258ce1d70c44e6a6d72 commit 52d4f5b16a150c7d6227f258ce1d70c44e6a6d72 Author: John Baldwin <jhb@FreeBSD.org> AuthorDate: 2022-09-29 22:05:22 +0000 Commit: Doug Moore <dougm@FreeBSD.org> CommitDate: 2022-10-12 02:47:45 +0000 rb_tree: Use void casts for RB_AUGMENT_CHECK with unused return value. Reviewed by: dougm Reported by: GCC -Wunused-value Differential Revision: https://reviews.freebsd.org/D36778 (cherry picked from commit 1aef571102e2d56b062916d69d962e8eeecfd1bf) --- sys/sys/tree.h | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/sys/sys/tree.h b/sys/sys/tree.h index 10f559f516d2..47cd14d6691b 100644 --- a/sys/sys/tree.h +++ b/sys/sys/tree.h @@ -603,8 +603,8 @@ name##_RB_INSERT_COLOR(struct name *head, \ * so update augmentation for them. \ */ \ if (elm != child) \ - RB_AUGMENT_CHECK(elm); \ - RB_AUGMENT_CHECK(parent); \ + (void)RB_AUGMENT_CHECK(elm); \ + (void)RB_AUGMENT_CHECK(parent); \ return (child); \ } while ((parent = gpar) != NULL); \ return (NULL); \ @@ -734,7 +734,7 @@ name##_RB_REMOVE_COLOR(struct name *head, \ * augmentation for it. \ */ \ if (sib != elm) \ - RB_AUGMENT_CHECK(sib); \ + (void)RB_AUGMENT_CHECK(sib); \ return (parent); \ } while (elm = parent, (parent = gpar) != NULL); \ return (NULL); \ @@ -793,8 +793,8 @@ name##_RB_REMOVE(struct name *head, struct type *out) \ * changed subtrees, so update augmentation for \ * them if AUGMENT_WALK didn't. \ */ \ - RB_AUGMENT_CHECK(opar); \ - RB_AUGMENT_CHECK(RB_PARENT(opar, field)); \ + (void)RB_AUGMENT_CHECK(opar); \ + (void)RB_AUGMENT_CHECK(RB_PARENT(opar, field)); \ } \ } \ return (out); \ @@ -830,7 +830,7 @@ name##_RB_INSERT(struct name *head, struct type *elm) \ * changed subtree, so update augmentation for it if \ * AUGMENT_WALK didn't. \ */ \ - RB_AUGMENT_CHECK(tmp); \ + (void)RB_AUGMENT_CHECK(tmp); \ return (NULL); \ }