summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2021-01-28 06:20:23 (GMT)
committerJunio C Hamano <gitster@pobox.com>2021-01-28 20:03:26 (GMT)
commit8380dcd700e80cd22745bcffb3625f90f943950c (patch)
treea4ab3fcfe1f8b98af53592eaedd64f7b07eb4dff
parent45ee13b942b26925b33d827bda2856e1ed0af0b7 (diff)
downloadgit-8380dcd700e80cd22745bcffb3625f90f943950c.zip
git-8380dcd700e80cd22745bcffb3625f90f943950c.tar.gz
git-8380dcd700e80cd22745bcffb3625f90f943950c.tar.bz2
oid_pos(): access table through const pointers
When we are looking up an oid in an array, we obviously don't need to write to the array. Let's mark it as const in the function interfaces, as well as in the local variables we use to derference the void pointer (note a few cases use pointers-to-pointers, so we mark everything const). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--builtin/name-rev.c4
-rw-r--r--commit-graph.c4
-rw-r--r--commit.c4
-rw-r--r--hash-lookup.c2
-rw-r--r--hash-lookup.h4
-rw-r--r--oid-array.c4
-rw-r--r--pack-bitmap-write.c4
7 files changed, 13 insertions, 13 deletions
diff --git a/builtin/name-rev.c b/builtin/name-rev.c
index 27138fd..b221d30 100644
--- a/builtin/name-rev.c
+++ b/builtin/name-rev.c
@@ -390,9 +390,9 @@ static void name_tips(void)
}
}
-static const struct object_id *nth_tip_table_ent(size_t ix, void *table_)
+static const struct object_id *nth_tip_table_ent(size_t ix, const void *table_)
{
- struct tip_table_entry *table = table_;
+ const struct tip_table_entry *table = table_;
return &table[ix].oid;
}
diff --git a/commit-graph.c b/commit-graph.c
index 248f1ef..b09fce5 100644
--- a/commit-graph.c
+++ b/commit-graph.c
@@ -1012,9 +1012,9 @@ static int write_graph_chunk_oids(struct hashfile *f,
return 0;
}
-static const struct object_id *commit_to_oid(size_t index, void *table)
+static const struct object_id *commit_to_oid(size_t index, const void *table)
{
- struct commit **commits = table;
+ const struct commit * const *commits = table;
return &commits[index]->object.oid;
}
diff --git a/commit.c b/commit.c
index 39eab5b..fd2831d 100644
--- a/commit.c
+++ b/commit.c
@@ -105,9 +105,9 @@ static timestamp_t parse_commit_date(const char *buf, const char *tail)
return parse_timestamp(dateptr, NULL, 10);
}
-static const struct object_id *commit_graft_oid_access(size_t index, void *table)
+static const struct object_id *commit_graft_oid_access(size_t index, const void *table)
{
- struct commit_graft **commit_graft_table = table;
+ const struct commit_graft * const *commit_graft_table = table;
return &commit_graft_table[index]->oid;
}
diff --git a/hash-lookup.c b/hash-lookup.c
index d15bb34..b98ed5e 100644
--- a/hash-lookup.c
+++ b/hash-lookup.c
@@ -50,7 +50,7 @@ static uint32_t take2(const struct object_id *oid, size_t ofs)
* The oid of element i (between 0 and nr - 1) should be returned
* by "fn(i, table)".
*/
-int oid_pos(const struct object_id *oid, void *table, size_t nr,
+int oid_pos(const struct object_id *oid, const void *table, size_t nr,
oid_access_fn fn)
{
size_t hi = nr;
diff --git a/hash-lookup.h b/hash-lookup.h
index 7b3ecad..dbd71eb 100644
--- a/hash-lookup.h
+++ b/hash-lookup.h
@@ -1,10 +1,10 @@
#ifndef HASH_LOOKUP_H
#define HASH_LOOKUP_H
-typedef const struct object_id *oid_access_fn(size_t index, void *table);
+typedef const struct object_id *oid_access_fn(size_t index, const void *table);
int oid_pos(const struct object_id *oid,
- void *table,
+ const void *table,
size_t nr,
oid_access_fn fn);
diff --git a/oid-array.c b/oid-array.c
index a19235a..73ba76e 100644
--- a/oid-array.c
+++ b/oid-array.c
@@ -22,9 +22,9 @@ void oid_array_sort(struct oid_array *array)
array->sorted = 1;
}
-static const struct object_id *oid_access(size_t index, void *table)
+static const struct object_id *oid_access(size_t index, const void *table)
{
- struct object_id *array = table;
+ const struct object_id *array = table;
return &array[index];
}
diff --git a/pack-bitmap-write.c b/pack-bitmap-write.c
index f21259d..88d9e69 100644
--- a/pack-bitmap-write.c
+++ b/pack-bitmap-write.c
@@ -610,9 +610,9 @@ static inline void dump_bitmap(struct hashfile *f, struct ewah_bitmap *bitmap)
die("Failed to write bitmap index");
}
-static const struct object_id *oid_access(size_t pos, void *table)
+static const struct object_id *oid_access(size_t pos, const void *table)
{
- struct pack_idx_entry **index = table;
+ const struct pack_idx_entry * const *index = table;
return &index[pos]->oid;
}