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_thread_time.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_thread_time.h')
| -rw-r--r-- | src/nxt_thread_time.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/nxt_thread_time.h b/src/nxt_thread_time.h index 7c133b4b..11ab0cd7 100644 --- a/src/nxt_thread_time.h +++ b/src/nxt_thread_time.h @@ -45,7 +45,7 @@ typedef struct { struct tm gmtime; struct tm localtime; - uint32_t no_cache; /* 1 bit */ + nxt_bool_t no_cache; /* * The flag indicating a signal state of a thread. |