reftable/block: use `size_t` to track restart point index
The function `block_reader_restart_offset()` gets the offset of the `i`th restart point. `i` is a signed integer though, which is certainly not the correct type to track indices like this. Furthermore, both callers end up passing a `size_t`. Refactor the code to use a `size_t` instead. Signed-off-by: Patrick Steinhardt <ps@pks.im> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Patrick Steinhardt committed
May 13, 2024 at 10:47 UTC
d537ce6b9ed75d50c6b8ad071439b06c1b70c5f8
1 file changed
+2
-2
reftable/block.c
+2
-2
@@ -326,9 +326,9 @@ int block_reader_first_key(const struct block_reader *br, struct strbuf *key)
326
return 0;
327
}
328
329
-static uint32_t block_reader_restart_offset(const struct block_reader *br, int i)
329
+static uint32_t block_reader_restart_offset(const struct block_reader *br, size_t idx)
330
{
331
- return get_be24(br->restart_bytes + 3 * i);
331
+ return get_be24(br->restart_bytes + 3 * idx);
332
}
333
334
void block_iter_seek_start(struct block_iter *it, const struct block_reader *br)