fix: -Wsingle-bit-bitfield-constant-conversion with clang16

We get the following warning with Clang 16:

  lttng-ust-abi.c:558:38: warning: implicit truncation from 'int' to a one-bit wide bit-field changes value from 1 to -1 [-Wsingle-bit-bitfield-constant-conversion]
         lttng_chan_buf->priv->parent.tstate = 1;

My understanding is that there is no bug because we only check if the
values are zero or not, so we can silence the warning by making the
variables unsigned.

Change-Id: Ic4e02164d5adf4271fa24e5b13e5d320ae19de2e
Signed-off-by: Michael Jeanson <mjeanson@efficios.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/src/common/events.h b/src/common/events.h
index 0c2d3136014bcb7af0a0d11186a3ada16658ca07..a145024969b0dd41758446cd0113c5638010a915 100644 (file)
--- a/src/common/events.h
+++ b/src/common/events.h
@@ -343,9 +343,9 @@ struct lttng_ust_session_private {
 
        struct lttng_ust_event_ht events_ht;    /* ht of events */
        void *owner;                            /* object owner */
-       int tstate:1;                           /* Transient enable state */
+       unsigned int tstate:1;                  /* Transient enable state */
 
-       int statedump_pending:1;
+       unsigned int statedump_pending:1;
 
        struct lttng_ust_enum_ht enums_ht;      /* ht of enumerations */
        struct cds_list_head enums_head;
@@ -393,7 +393,7 @@ struct lttng_ust_channel_common_private {
        struct lttng_ust_channel_common *pub;   /* Public channel interface */
 
        int objd;                               /* Object associated with channel. */
-       int tstate:1;                           /* Transient enable state */
+       unsigned int tstate:1;                  /* Transient enable state */
 };
 
 struct lttng_ust_channel_buffer_private {