threads / patch / 6655

patch, 2 partsfix git-rev-parse for ...@{0}

Subject: [PATCH 1.5/2] fix git-rev-parse for ...@{0}

## tl;dr

One message between Feb 4, 2007 and Feb 4, 2007. Diffs are folded; open one to read it.

replies: 0people: 1as markdown or json

Nicolas Pitre· Feb 4, 2007, 03:56 UTC · lore
My patch enabling HEAD@{...} broke the @{0} case.
Signed-off-by: Nicolas Pitre <nico@cam.org>
---
 sha1_name.c |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)
Show changes to sha1_name.c +5 −3
diff --git a/sha1_name.c b/sha1_name.c
index 9841b05..d77f770 100644
--- a/sha1_name.c
+++ b/sha1_name.c
@@ -267,7 +267,7 @@ int dwim_ref(const char *str, int len, unsigned char *sha1, char **ref)
 	return refs_found;
 }
 
-static int dwim_log(const char *str, int len, char **log)
+static int dwim_log(const char *str, int len, unsigned char *sha1, char **log)
 {
 	const char **p;
 	int logs_found = 0;
@@ -278,8 +278,10 @@ static int dwim_log(const char *str, int len, char **log)
 		char *path = mkpath(*p, len, str);
 		if (!stat(git_path("logs/%s", path), &st) &&
 		    S_ISREG(st.st_mode)) {
-			if (!logs_found++)
+			if (!logs_found++) {
 				*log = xstrdup(path);
+				resolve_ref(path, sha1, 0, NULL);
+			}
 			if (!warn_ambiguous_refs)
 				break;
 		}
@@ -317,7 +319,7 @@ static int get_sha1_basic(const char *str, int len, unsigned char *sha1)
 		/* allow "@{...}" to mean the current branch reflog */
 		refs_found = dwim_ref("HEAD", 4, sha1, &real_ref);
 	} else if (reflog_len)
-		refs_found = dwim_log(str, len, &real_ref);
+		refs_found = dwim_log(str, len, sha1, &real_ref);
 	else
 		refs_found = dwim_ref(str, len, sha1, &real_ref);
 
-- 
1.5.0.rc2.652.g4dfde-dirty

← back to recent threads