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_sendbuf.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_sendbuf.h')
-rw-r--r-- | src/nxt_sendbuf.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/nxt_sendbuf.h b/src/nxt_sendbuf.h index fcbe1a25..4bc33e4e 100644 --- a/src/nxt_sendbuf.h +++ b/src/nxt_sendbuf.h @@ -44,10 +44,10 @@ typedef struct { size_t size; size_t limit; - uint8_t ready; /* 1 bit */ - uint8_t once; /* 1 bit */ - uint8_t sync; /* 1 bit */ - uint8_t last; /* 1 bit */ + nxt_bool_t ready; + nxt_bool_t once; + nxt_bool_t sync; + nxt_bool_t last; } nxt_sendbuf_t; @@ -57,8 +57,8 @@ typedef struct { nxt_uint_t niov; uint32_t nmax; - uint8_t sync; /* 1 bit */ - uint8_t last; /* 1 bit */ + nxt_bool_t sync; + nxt_bool_t last; uint8_t limit_reached; uint8_t nmax_reached; |