From mboxrd@z Thu Jan 1 00:00:00 1970 From: mhagger@alum.mit.edu Subject: [PATCH v3 04/14] refs: rename parameters result -> sha1 Date: Fri, 28 Oct 2011 13:14:31 +0200 Message-ID: <1319800481-15138-5-git-send-email-mhagger@alum.mit.edu> References: <1319800481-15138-1-git-send-email-mhagger@alum.mit.edu> Cc: git@vger.kernel.org, Jeff King , Drew Northup , Jakub Narebski , Heiko Voigt , Johan Herland , Julian Phillips , Michael Haggerty To: Junio C Hamano X-From: git-owner@vger.kernel.org Fri Oct 28 13:15:51 2011 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1RJkPd-00058s-Ga for gcvg-git-2@lo.gmane.org; Fri, 28 Oct 2011 13:15:49 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755413Ab1J1LPm (ORCPT ); Fri, 28 Oct 2011 07:15:42 -0400 Received: from mail.berlin.jpk.com ([212.222.128.130]:54700 "EHLO mail.berlin.jpk.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755260Ab1J1LPG (ORCPT ); Fri, 28 Oct 2011 07:15:06 -0400 Received: from michael.berlin.jpk.com ([192.168.100.152]) by mail.berlin.jpk.com with esmtp (Exim 4.50) id 1RJkIZ-00076m-24; Fri, 28 Oct 2011 13:08:31 +0200 X-Mailer: git-send-email 1.7.7 In-Reply-To: <1319800481-15138-1-git-send-email-mhagger@alum.mit.edu> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: From: Michael Haggerty Try consistently to use the name "sha1" for parameters to which a SHA1 will be stored. Signed-off-by: Michael Haggerty --- refs.c | 16 ++++++++-------- refs.h | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git a/refs.c b/refs.c index 7a34e4b..564070c 100644 --- a/refs.c +++ b/refs.c @@ -402,7 +402,7 @@ static struct ref_array *get_loose_refs(const char *submodule) * the packed-refs file for the submodule. */ static int resolve_gitlink_packed_ref(char *name, int pathlen, - const char *refname, unsigned char *result) + const char *refname, unsigned char *sha1) { int retval = -1; struct ref_entry *ref; @@ -415,14 +415,14 @@ static int resolve_gitlink_packed_ref(char *name, int pathlen, array = get_packed_refs(name); ref = search_ref_array(array, refname); if (ref != NULL) { - memcpy(result, ref->sha1, 20); + memcpy(sha1, ref->sha1, 20); retval = 0; } return retval; } static int resolve_gitlink_ref_recursive(char *name, int pathlen, - const char *refname, unsigned char *result, + const char *refname, unsigned char *sha1, int recursion) { int fd, len = strlen(refname); @@ -433,7 +433,7 @@ static int resolve_gitlink_ref_recursive(char *name, int pathlen, memcpy(name + pathlen, refname, len+1); fd = open(name, O_RDONLY); if (fd < 0) - return resolve_gitlink_packed_ref(name, pathlen, refname, result); + return resolve_gitlink_packed_ref(name, pathlen, refname, sha1); len = read(fd, buffer, sizeof(buffer)-1); close(fd); @@ -444,7 +444,7 @@ static int resolve_gitlink_ref_recursive(char *name, int pathlen, buffer[len] = 0; /* Was it a detached head or an old-fashioned symlink? */ - if (!get_sha1_hex(buffer, result)) + if (!get_sha1_hex(buffer, sha1)) return 0; /* Symref? */ @@ -454,10 +454,10 @@ static int resolve_gitlink_ref_recursive(char *name, int pathlen, while (isspace(*p)) p++; - return resolve_gitlink_ref_recursive(name, pathlen, p, result, recursion+1); + return resolve_gitlink_ref_recursive(name, pathlen, p, sha1, recursion+1); } -int resolve_gitlink_ref(const char *path, const char *refname, unsigned char *result) +int resolve_gitlink_ref(const char *path, const char *refname, unsigned char *sha1) { int len = strlen(path), retval; char *gitdir; @@ -481,7 +481,7 @@ int resolve_gitlink_ref(const char *path, const char *refname, unsigned char *re } gitdir[len] = '/'; gitdir[++len] = '\0'; - retval = resolve_gitlink_ref_recursive(gitdir, len, refname, result, 0); + retval = resolve_gitlink_ref_recursive(gitdir, len, refname, sha1, 0); free(gitdir); return retval; } diff --git a/refs.h b/refs.h index e36bd03..4c5d570 100644 --- a/refs.h +++ b/refs.h @@ -134,7 +134,7 @@ extern char *shorten_unambiguous_ref(const char *refname, int strict); extern int rename_ref(const char *oldref, const char *newref, const char *logmsg); /** resolve ref in nested "gitlink" repository */ -extern int resolve_gitlink_ref(const char *name, const char *refname, unsigned char *result); +extern int resolve_gitlink_ref(const char *name, const char *refname, unsigned char *sha1); /** lock a ref and then write its file */ enum action_on_err { MSG_ON_ERR, DIE_ON_ERR, QUIET_ON_ERR }; -- 1.7.7