diff options
| author | Andrew Clayton <a.clayton@nginx.com> | 2023-03-27 22:41:53 +0100 |
|---|---|---|
| committer | Andrew Clayton <a.clayton@nginx.com> | 2023-03-28 23:15:28 +0100 |
| commit | aa0009f8dcecfefd3c123baa316c2f86d3525f8c (patch) | |
| tree | 34f80dfa052a388fd38da7a5149519da61b30526 /src/nxt_event_engine.h | |
| parent | c1303660eda80c5e17fde06bb8f17715e7be049b (diff) | |
| download | unit-aa0009f8dcecfefd3c123baa316c2f86d3525f8c.tar.gz unit-aa0009f8dcecfefd3c123baa316c2f86d3525f8c.tar.bz2 | |
Convert uint32_t struct boolean members to nxt_bool_t.
This commit complements the earlier one that changed a bunch of uint8_t
struct members to nxt_bool_t (aka uint8_t).
It's not entirely clear why these were specifically done as uint32_t's.
I've checked these structure layouts with pahole(1) after this change on
both 32 and 64bit and the layouts haven't changed.
We may get a 3 or 7 byte hole after the nxt_bool_t or 3 bytes of padding
at the end of the structure, but the structure members are still at the
same offsets and the structure sizes have remained the same.
Reviewed-by: Alejandro Colomar <alx@nginx.com>
Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'src/nxt_event_engine.h')
| -rw-r--r-- | src/nxt_event_engine.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/nxt_event_engine.h b/src/nxt_event_engine.h index 8773acee..850a7854 100644 --- a/src/nxt_event_engine.h +++ b/src/nxt_event_engine.h @@ -329,7 +329,7 @@ extern const nxt_event_interface_t nxt_poll_engine; typedef struct { int nfds; - uint32_t update_nfds; /* 1 bit */ + nxt_bool_t update_nfds; nxt_fd_event_t **events; |