diff options
| author | Junio C Hamano <gitster@pobox.com> | 2025-01-28 13:02:24 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2025-01-28 13:02:24 -0800 |
| commit | a17fd7dd3ada289df8e44e64ff52a5d886dfee95 (patch) | |
| tree | bbc7b3f5eb263b1f40490bf896f38430e7f40c09 /reftable/writer.c | |
| parent | 73e055d71ea39c54e78b6e9a28ea0d8e7999a5cb (diff) | |
| parent | 33319b0976ff9975e7509b6096887370146893f4 (diff) | |
| download | git-a17fd7dd3ada289df8e44e64ff52a5d886dfee95.tar.xz | |
Merge branch 'ps/reftable-sign-compare'
The reftable/ library code has been made -Wsign-compare clean.
* ps/reftable-sign-compare:
reftable: address trivial -Wsign-compare warnings
reftable/blocksource: adjust `read_block()` to return `ssize_t`
reftable/blocksource: adjust type of the block length
reftable/block: adjust type of the restart length
reftable/block: adapt header and footer size to return a `size_t`
reftable/basics: adjust `hash_size()` to return `uint32_t`
reftable/basics: adjust `common_prefix_size()` to return `size_t`
reftable/record: handle overflows when decoding varints
reftable/record: drop unused `print` function pointer
meson: stop disabling -Wsign-compare
Diffstat (limited to 'reftable/writer.c')
| -rw-r--r-- | reftable/writer.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/reftable/writer.c b/reftable/writer.c index 740c98038e..91d6629486 100644 --- a/reftable/writer.c +++ b/reftable/writer.c @@ -577,7 +577,7 @@ static int writer_finish_section(struct reftable_writer *w) struct common_prefix_arg { struct reftable_buf *last; - int max; + size_t max; }; static void update_common(void *void_arg, void *key) @@ -585,10 +585,9 @@ static void update_common(void *void_arg, void *key) struct common_prefix_arg *arg = void_arg; struct obj_index_tree_node *entry = key; if (arg->last) { - int n = common_prefix_size(&entry->hash, arg->last); - if (n > arg->max) { + size_t n = common_prefix_size(&entry->hash, arg->last); + if (n > arg->max) arg->max = n; - } } arg->last = &entry->hash; } |
