diff options
author | Derrick Stolee <derrickstolee@github.com> | 2022-04-25 21:27:14 +0300 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2022-04-25 21:31:13 +0300 |
commit | 11f9e8de3d1a3d484c452141f1fcdbd707457ec0 (patch) | |
tree | a5dad66bc4f77b7137b129f9bc79f0ed43e7d866 | |
parent | b56166ca57678795b4159b6a48b64fbacb93e73d (diff) |
cache: use const char * for get_object_directory()
The get_object_directory() method returns the exact string stored at
the_repository->objects->odb->path. The return type of "char *" implies
that the caller must keep track of the buffer and free() it when
complete. This causes significant problems later when the ODB is
accessed.
Use "const char *" as the return type to avoid this confusion. There are
no current callers that care about the non-const definition.
Signed-off-by: Derrick Stolee <derrickstolee@github.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r-- | cache.h | 2 | ||||
-rw-r--r-- | environment.c | 2 |
2 files changed, 2 insertions, 2 deletions
@@ -567,7 +567,7 @@ extern char *git_work_tree_cfg; int is_inside_work_tree(void); const char *get_git_dir(void); const char *get_git_common_dir(void); -char *get_object_directory(void); +const char *get_object_directory(void); char *get_index_file(void); char *get_graft_file(struct repository *r); void set_git_dir(const char *path, int make_realpath); diff --git a/environment.c b/environment.c index fd0501e77a..4e49ee0d60 100644 --- a/environment.c +++ b/environment.c @@ -271,7 +271,7 @@ const char *get_git_work_tree(void) return the_repository->worktree; } -char *get_object_directory(void) +const char *get_object_directory(void) { if (!the_repository->objects->odb) BUG("git environment hasn't been setup"); |