diff options
author | Junio C Hamano <gitster@pobox.com> | 2019-02-07 09:05:27 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2019-02-07 09:05:28 +0300 |
commit | 3a14fdec88125c4fd093f938cf406231b932e3a2 (patch) | |
tree | 1b67f5143fdd6b3be964a2e57ac65c557d1fe566 | |
parent | 257507a35e1d13d8849bbdf9324a060232a82bdf (diff) | |
parent | 33de80b1d50773f79e380eff33b4fc5c8ae1dd25 (diff) |
Merge branch 'sl/const'
Code cleanup.
* sl/const:
various: tighten constness of some local variables
-rw-r--r-- | builtin/diff.c | 2 | ||||
-rw-r--r-- | builtin/pack-objects.c | 14 | ||||
-rw-r--r-- | builtin/pack-redundant.c | 4 | ||||
-rw-r--r-- | pack-revindex.c | 10 |
4 files changed, 15 insertions, 15 deletions
diff --git a/builtin/diff.c b/builtin/diff.c index 74351a5757..9f6109224b 100644 --- a/builtin/diff.c +++ b/builtin/diff.c @@ -103,7 +103,7 @@ static int builtin_diff_blobs(struct rev_info *revs, int argc, const char **argv, struct object_array_entry **blob) { - unsigned mode = canon_mode(S_IFREG | 0644); + const unsigned mode = canon_mode(S_IFREG | 0644); if (argc > 1) usage(builtin_diff_usage); diff --git a/builtin/pack-objects.c b/builtin/pack-objects.c index 68e2e66db2..a9fac7c128 100644 --- a/builtin/pack-objects.c +++ b/builtin/pack-objects.c @@ -1902,10 +1902,10 @@ static int type_size_sort(const void *_a, const void *_b) { const struct object_entry *a = *(struct object_entry **)_a; const struct object_entry *b = *(struct object_entry **)_b; - enum object_type a_type = oe_type(a); - enum object_type b_type = oe_type(b); - unsigned long a_size = SIZE(a); - unsigned long b_size = SIZE(b); + const enum object_type a_type = oe_type(a); + const enum object_type b_type = oe_type(b); + const unsigned long a_size = SIZE(a); + const unsigned long b_size = SIZE(b); if (a_type > b_type) return -1; @@ -1920,7 +1920,7 @@ static int type_size_sort(const void *_a, const void *_b) if (a->preferred_base < b->preferred_base) return 1; if (use_delta_islands) { - int island_cmp = island_delta_cmp(&a->idx.oid, &b->idx.oid); + const int island_cmp = island_delta_cmp(&a->idx.oid, &b->idx.oid); if (island_cmp) return island_cmp; } @@ -2167,7 +2167,7 @@ static unsigned int check_delta_limit(struct object_entry *me, unsigned int n) struct object_entry *child = DELTA_CHILD(me); unsigned int m = n; while (child) { - unsigned int c = check_delta_limit(child, n + 1); + const unsigned int c = check_delta_limit(child, n + 1); if (m < c) m = c; child = DELTA_SIBLING(child); @@ -2222,7 +2222,7 @@ static void find_deltas(struct object_entry **list, unsigned *list_size, while (window_memory_limit && mem_usage > window_memory_limit && count > 1) { - uint32_t tail = (idx + window - count) % window; + const uint32_t tail = (idx + window - count) % window; mem_usage -= free_unpacked(array + tail); count--; } diff --git a/builtin/pack-redundant.c b/builtin/pack-redundant.c index cf9a9aabd4..11bc514566 100644 --- a/builtin/pack-redundant.c +++ b/builtin/pack-redundant.c @@ -166,7 +166,7 @@ redo_from_start: l = (hint == NULL) ? list->front : hint; prev = NULL; while (l) { - int cmp = oidcmp(l->oid, oid); + const int cmp = oidcmp(l->oid, oid); if (cmp > 0) /* not in list, since sorted */ return prev; if (!cmp) { /* found */ @@ -264,7 +264,7 @@ static void cmp_two_packs(struct pack_list *p1, struct pack_list *p2) while (p1_off < p1->pack->num_objects * p1_step && p2_off < p2->pack->num_objects * p2_step) { - int cmp = hashcmp(p1_base + p1_off, p2_base + p2_off); + const int cmp = hashcmp(p1_base + p1_off, p2_base + p2_off); /* cmp ~ p1 - p2 */ if (cmp == 0) { p1_hint = llist_sorted_remove(p1->unique_objects, diff --git a/pack-revindex.c b/pack-revindex.c index 3c58784a5f..50891f77a2 100644 --- a/pack-revindex.c +++ b/pack-revindex.c @@ -119,7 +119,7 @@ static void sort_revindex(struct revindex_entry *entries, unsigned n, off_t max) */ static void create_pack_revindex(struct packed_git *p) { - unsigned num_ent = p->num_objects; + const unsigned num_ent = p->num_objects; unsigned i; const char *index = p->index_data; const unsigned hashsz = the_hash_algo->rawsz; @@ -132,7 +132,7 @@ static void create_pack_revindex(struct packed_git *p) (uint32_t *)(index + 8 + p->num_objects * (hashsz + 4)); const uint32_t *off_64 = off_32 + p->num_objects; for (i = 0; i < num_ent; i++) { - uint32_t off = ntohl(*off_32++); + const uint32_t off = ntohl(*off_32++); if (!(off & 0x80000000)) { p->revindex[i].offset = off; } else { @@ -143,7 +143,7 @@ static void create_pack_revindex(struct packed_git *p) } } else { for (i = 0; i < num_ent; i++) { - uint32_t hl = *((uint32_t *)(index + (hashsz + 4) * i)); + const uint32_t hl = *((uint32_t *)(index + (hashsz + 4) * i)); p->revindex[i].offset = ntohl(hl); p->revindex[i].nr = i; } @@ -168,10 +168,10 @@ int find_revindex_position(struct packed_git *p, off_t ofs) { int lo = 0; int hi = p->num_objects + 1; - struct revindex_entry *revindex = p->revindex; + const struct revindex_entry *revindex = p->revindex; do { - unsigned mi = lo + (hi - lo) / 2; + const unsigned mi = lo + (hi - lo) / 2; if (revindex[mi].offset == ofs) { return mi; } else if (ofs < revindex[mi].offset) |