diff options
| author | Andrew Clayton <a.clayton@nginx.com> | 2023-03-23 20:16:08 +0000 |
|---|---|---|
| committer | Andrew Clayton <a.clayton@nginx.com> | 2023-03-28 23:15:28 +0100 |
| commit | c1303660eda80c5e17fde06bb8f17715e7be049b (patch) | |
| tree | 6604e5d93ef42ce3ced9c5cf8d4dc2ee24bd5170 /src/nxt_conn.h | |
| parent | f20172e5959d2029fdc7f084a332a748563dd50e (diff) | |
| download | unit-c1303660eda80c5e17fde06bb8f17715e7be049b.tar.gz unit-c1303660eda80c5e17fde06bb8f17715e7be049b.tar.bz2 | |
Convert uint8_t struct boolean members to nxt_bool_t.
Replace the usage of uint8_t in structures to represent boolean values
with our nxt_bool_t type.
This will result in no change in structure layout as the nxt_bool_t is
now a uint8_t, same as what it's replacing.
Even though it's essentially the same type, it makes it much clearer as
to what its purpose is.
This was largely done with the following script from Alex, with some
manual conversions
$ grep -rl 'uint8_t.*1 bit' src/ \
| xargs sed -i '/uint8_t.*1 bit/{s/uint8_t /nxt_bool_t /;s/; *\/\*.*/;/}'
This doesn't convert the non-uint8_t booleans, they will be handled
separately.
Reviewed-by: Alejandro Colomar <alx@nginx.com>
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'src/nxt_conn.h')
| -rw-r--r-- | src/nxt_conn.h | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/nxt_conn.h b/src/nxt_conn.h index 5717d3c9..9301c809 100644 --- a/src/nxt_conn.h +++ b/src/nxt_conn.h @@ -157,17 +157,17 @@ struct nxt_conn_s { nxt_sockaddr_t *local; const char *action; - uint8_t block_read; /* 1 bit */ - uint8_t block_write; /* 1 bit */ - uint8_t delayed; /* 1 bit */ - uint8_t idle; /* 1 bit */ + nxt_bool_t block_read; + nxt_bool_t block_write; + nxt_bool_t delayed; + nxt_bool_t idle; #define NXT_CONN_SENDFILE_OFF 0 #define NXT_CONN_SENDFILE_ON 1 #define NXT_CONN_SENDFILE_UNSET 3 uint8_t sendfile; /* 2 bits */ - uint8_t tcp_nodelay; /* 1 bit */ + nxt_bool_t tcp_nodelay; nxt_queue_link_t link; }; @@ -339,8 +339,8 @@ typedef struct { nxt_msec_t client_write_timeout; nxt_msec_t peer_write_timeout; - uint8_t connected; /* 1 bit */ - uint8_t delayed; /* 1 bit */ + nxt_bool_t connected; + nxt_bool_t delayed; uint8_t retries; /* 8 bits */ uint8_t retain; /* 2 bits */ |