diff options
| author | Michael Smith <mikesmiffy128@gmail.com> | 2024-08-03 23:40:31 +0100 |
|---|---|---|
| committer | Michael Smith <mikesmiffy128@gmail.com> | 2024-08-23 20:37:37 +0100 |
| commit | 83da606072ce272eb053d4e1497d77e647cfecae (patch) | |
| tree | 71d0110881ff8685184c5f4ab720cc8d49c24678 /src/bitbuf.h | |
| parent | acbd30e0427b16f885f96aed59881ec04eff25bc (diff) | |
Revise syntax macros and add a ton of branch hints
My new programming style is branch hints. All non-confusing branches
must be hinted when I can be bothered. It's faster, sometimes, maybe.
Also, start trying to use more signed sizes in at least some of the
places where it makes sense. Unsigned sizes are surprisingly
error-prone!
Diffstat (limited to 'src/bitbuf.h')
| -rw-r--r-- | src/bitbuf.h | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/bitbuf.h b/src/bitbuf.h index 404dc9d..9e0fe19 100644 --- a/src/bitbuf.h +++ b/src/bitbuf.h @@ -1,5 +1,5 @@ /* - * Copyright © 2023 Michael Smith <mikesmiffy128@gmail.com> + * Copyright © 2024 Michael Smith <mikesmiffy128@gmail.com> * * Permission to use, copy, modify, and/or distribute this software for any * purpose with or without fee is hereby granted, provided that the above @@ -77,11 +77,12 @@ static inline void bitbuf_appendbuf(struct bitbuf *bb, const char *buf, // shift the stored value (if it were big endian, the shift would have // to be the other way, or something) _bitbuf_append(bb, *p >> (unalign << 3), (bitbuf_align - unalign) << 3); - buf += sizeof(bitbuf_cell) - unalign; + buf += (int)sizeof(bitbuf_cell) - unalign; len -= unalign; } bitbuf_cell *aligned = (bitbuf_cell *)buf; - for (; len >= sizeof(bitbuf_cell); len -= sizeof(bitbuf_cell), ++aligned) { + for (; len >= (int)sizeof(bitbuf_cell); len -= (int)sizeof(bitbuf_cell), + ++aligned) { _bitbuf_append(bb, *aligned, bitbuf_cell_bits); } // unaligned end bytes |