diff options
| author | Patrick Steinhardt <ps@pks.im> | 2024-05-13 10:47:01 +0200 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2024-05-13 17:04:16 -0700 |
| commit | d537ce6b9ed75d50c6b8ad071439b06c1b70c5f8 (patch) | |
| tree | 86eaf8f220304d474df3ead1403ebc0703927ff5 | |
| parent | 0f3415f1f8478b05e64db11eb8aaa2915e48fef6 (diff) | |
| download | git-d537ce6b9ed75d50c6b8ad071439b06c1b70c5f8.tar.gz | |
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>
| -rw-r--r-- | reftable/block.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/reftable/block.c b/reftable/block.c index 5942cb4053..00030eee06 100644 --- a/reftable/block.c +++ b/reftable/block.c @@ -326,9 +326,9 @@ int block_reader_first_key(const struct block_reader *br, struct strbuf *key) return 0; } -static uint32_t block_reader_restart_offset(const struct block_reader *br, int i) +static uint32_t block_reader_restart_offset(const struct block_reader *br, size_t idx) { - return get_be24(br->restart_bytes + 3 * i); + return get_be24(br->restart_bytes + 3 * idx); } void block_iter_seek_start(struct block_iter *it, const struct block_reader *br) |