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_unit.c | |
| 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_unit.c')
| -rw-r--r-- | src/nxt_unit.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/nxt_unit.c b/src/nxt_unit.c index e1b1897a..9d696f5e 100644 --- a/src/nxt_unit.c +++ b/src/nxt_unit.c @@ -222,8 +222,8 @@ struct nxt_unit_recv_msg_s { nxt_pid_t pid; nxt_port_id_t reply_port; - uint8_t last; /* 1 bit */ - uint8_t mmap; /* 1 bit */ + nxt_bool_t last; + nxt_bool_t mmap; void *start; uint32_t size; @@ -319,8 +319,8 @@ struct nxt_unit_ctx_impl_s { /* of nxt_unit_read_buf_t */ nxt_queue_t free_rbuf; - uint8_t online; /* 1 bit */ - uint8_t ready; /* 1 bit */ + nxt_bool_t online; + nxt_bool_t ready; uint8_t quit_param; nxt_unit_mmap_buf_t ctx_buf[2]; |