cocci: use MEMZERO_ARRAY() a bit more
Existing code in files that have been fairly stable trigger the "make coccicheck" suggestions due to the new check. Rewrite them to use MEMZERO_ARRAY() Signed-off-by: Junio C Hamano <gitster@pobox.com>
Junio C Hamano committed
Dec 13, 2025 at 10:46 UTC
8ea9492cf3505c379d1c573b02db90e6b480cc75
3 files changed
+6
-6
diffcore-delta.c
+2
-2
index ba6cbee76b..2de9e9ccff 100644
--- a/diffcore-delta.c
+++ b/diffcore-delta.c
@@ -56,7 +56,7 @@ static struct spanhash_top *spanhash_rehash(struct spanhash_top *orig)
st_mult(sizeof(struct spanhash), sz)));
new_spanhash->alloc_log2 = orig->alloc_log2 + 1;
new_spanhash->free = INITIAL_FREE(new_spanhash->alloc_log2);
- memset(new_spanhash->data, 0, sizeof(struct spanhash) * sz);
+ MEMZERO_ARRAY(new_spanhash->data, sz);
for (i = 0; i < osz; i++) {
struct spanhash *o = &(orig->data[i]);
int bucket;
@@ -135,7 +135,7 @@ static struct spanhash_top *hash_chars(struct repository *r,
st_mult(sizeof(struct spanhash), (size_t)1 << i)));
hash->alloc_log2 = i;
hash->free = INITIAL_FREE(i);
- memset(hash->data, 0, sizeof(struct spanhash) * ((size_t)1 << i));
+ MEMZERO_ARRAY(hash->data, ((size_t)1 << i));
n = 0;
accum1 = accum2 = 0;
linear-assignment.c
+2
-2
index 5416cbcf40..97b4f75058 100644
--- a/linear-assignment.c
+++ b/linear-assignment.c
@@ -20,8 +20,8 @@ void compute_assignment(int column_count, int row_count, int *cost,
int i, j, phase;
if (column_count < 2) {
- memset(column2row, 0, sizeof(int) * column_count);
- memset(row2column, 0, sizeof(int) * row_count);
+ MEMZERO_ARRAY(column2row, column_count);
+ MEMZERO_ARRAY(row2column, row_count);
return;
}
shallow.c
+2
-2
index d9cd4e219c..c20471cd7e 100644
--- a/shallow.c
+++ b/shallow.c
@@ -713,7 +713,7 @@ void assign_shallow_commits_to_refs(struct shallow_info *info,
if (used) {
int bitmap_size = DIV_ROUND_UP(pi.nr_bits, 32) * sizeof(uint32_t);
- memset(used, 0, sizeof(*used) * info->shallow->nr);
+ MEMZERO_ARRAY(used, info->shallow->nr);
for (i = 0; i < nr_shallow; i++) {
const struct commit *c = lookup_commit(the_repository,
&oid[shallow[i]]);
@@ -782,7 +782,7 @@ static void post_assign_shallow(struct shallow_info *info,
trace_printf_key(&trace_shallow, "shallow: post_assign_shallow\n");
if (ref_status)
- memset(ref_status, 0, sizeof(*ref_status) * info->ref->nr);
+ MEMZERO_ARRAY(ref_status, info->ref->nr);
/* Remove unreachable shallow commits from "theirs" */
for (i = dst = 0; i < info->nr_theirs; i++) {