[PATCH v3 1/2] fsck.c: modify fsck_ident() and fsck_commit()
- From
- Ashwin Jha <ajha.dev@gmail.com>
- Date
- Mar 24, 2014, 12:08 UTC
- Message-ID
- <1395662901-10617-1-git-send-email-ajha.dev@gmail.com>
In fsck_ident(): Replace argument char **ident with const char **ident In fsck_commit(): Replace char *buffer with const char *buffer
In both the cases, referenced memory addresses are not modified. So, it will be a good practice, to declare them as const.
Signed-off-by: Ashwin Jha <ajha.dev@gmail.com> ---
Change in fsck_commit() and fsck_ident() as per the discussion with Eric (follow at [1]). [1]: http://git.661346.n2.nabble.com/PATCH-Modify-fsck-commit-Replace-memcmp-by-skip-prefix-td7606321.html
fsck.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fsck.c b/fsck.c index 64bf279..b5f017f 100644 --- a/fsck.c +++ b/fsck.c @@ -243,7 +243,7 @@ static int fsck_tree(struct tree *item, int strict, fsck_error error_func) return retval; } -static int fsck_ident(char **ident, struct object *obj, fsck_error error_func) +static int fsck_ident(const char **ident, struct object *obj, fsck_error error_func) { char *end; @@ -284,7 +284,7 @@ static int fsck_ident(char **ident, struct object *obj, fsck_error error_func) static int fsck_commit(struct commit *commit, fsck_error error_func) { - char *buffer = commit->buffer; + const char *buffer = commit->buffer; unsigned char tree_sha1[20], sha1[20]; struct commit_graft *graft; int parents = 0;
-- 1.9.1