Skip to content

Minor comment clarifications and trivial rewrites#529

Merged
cperciva merged 1 commit intomasterfrom
misc
Jan 12, 2025
Merged

Minor comment clarifications and trivial rewrites#529
cperciva merged 1 commit intomasterfrom
misc

Conversation

@gperciva
Copy link
Member

No description provided.

cpusupport-ARM-AES.c: the info about _u32 was in a git commit message,
    but it's worth adding a 1-line comment about it.

cpusupport-X86-SSE42*.c: our STYLE says that we should never use
    (unsigned char).  That's what the definition of _mm_crc32_u8() uses,
    but the code in alg/crc32_sse42.c uses (uint8_t), so there's no
    point violating our STYLE by using (unsigned char) here.

network_ssl_compat.c: fix style of assignment & checking.

imalloc.h: split the "return (NULL)" cases from the actual malloc().
@cperciva cperciva merged commit d78d67e into master Jan 12, 2025
2 checks passed
@gperciva gperciva deleted the misc branch January 12, 2025 20:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants