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_http.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_http.h')
-rw-r--r-- | src/nxt_http.h | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/src/nxt_http.h b/src/nxt_http.h index ffd3f601..7739d9cd 100644 --- a/src/nxt_http.h +++ b/src/nxt_http.h @@ -124,8 +124,8 @@ typedef struct { nxt_http_status_t status:16; nxt_http_protocol_t protocol:8; /* 2 bits */ - uint8_t header_received; /* 1 bit */ - uint8_t closed; /* 1 bit */ + nxt_bool_t header_received; + nxt_bool_t closed; } nxt_http_peer_t; @@ -189,17 +189,17 @@ struct nxt_http_request_s { nxt_http_status_t status:16; - uint8_t log_route; /* 1 bit */ + nxt_bool_t log_route; uint8_t pass_count; /* 8 bits */ uint8_t app_target; nxt_http_protocol_t protocol:8; /* 2 bits */ - uint8_t tls; /* 1 bit */ - uint8_t logged; /* 1 bit */ - uint8_t header_sent; /* 1 bit */ - uint8_t inconsistent; /* 1 bit */ - uint8_t error; /* 1 bit */ - uint8_t websocket_handshake; /* 1 bit */ + nxt_bool_t tls; + nxt_bool_t logged; + nxt_bool_t header_sent; + nxt_bool_t inconsistent; + nxt_bool_t error; + nxt_bool_t websocket_handshake; }; @@ -288,7 +288,7 @@ struct nxt_http_forward_s { nxt_http_forward_header_t client_ip; nxt_http_forward_header_t protocol; nxt_http_route_addr_rule_t *source; - uint8_t recursive; /* 1 bit */ + nxt_bool_t recursive; }; |