Thread (83 messages) 83 messages, 2 authors, 2026-02-24

Re: [PATCH 02/17] midx: split `get_midx_checksum()` by adding `get_midx_hash()`

From: Taylor Blau <hidden>
Date: 2025-12-09 01:42:25

On Mon, Dec 08, 2025 at 07:25:05PM +0100, Patrick Steinhardt wrote:
If I didn't have the context of this patch series I would be wondering
what the actual difference between `get_midx_checksum()` and
`get_midx_hash()` is. The way the functions are named seems to rather
indicate that we talk about two different kinds of hashes, rather than
two different ways to encode them.

Would it maybe be preferable to call them `get_midx_checksum()` and
`get_midx_checksum_hex()`? While at it, we could go even further and
rename them to `midx_get_checksum()` and `midx_get_checksum_hex()` to
conform to our modern best practices.
Yeah, I think those are both reasonable suggestions; I'll apply those
locally, thanks!
Patrick
Thanks,
Taylor
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help