git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v5 0/7] refs backend preamble

From
Junio C Hamano <gitster@pobox.com>
Date
Jun 29, 2015, 19:11 UTC
Message-ID
<xmqqfv5a9vot.fsf@gitster.dls.corp.google.com>
In-Reply-To
<1435600468-21710-1-git-send-email-dturner@twopensource.com>

There might be other issues but from cursory read, at least the following needs to be split and squashed into patches that introduce them.

Otherwise, it was a very pleasant read.
Thanks.
 Documentation/git-reflog.txt | 4 ++--
 builtin/reflog.c             | 2 +-
 refs.c                       | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/Documentation/git-reflog.txt b/Documentation/git-reflog.txt
index 7ab2c42..2bf8aa6 100644
--- a/Documentation/git-reflog.txt
+++ b/Documentation/git-reflog.txt
@@ -23,8 +23,8 @@ depending on the subcommand:
 	[--dry-run] [--verbose] [--all | <refs>...]
 'git reflog delete' [--rewrite] [--updateref]
 	[--dry-run] [--verbose] ref@\{specifier\}...
-'git reflog create <refs>...
-'git reflog exists <ref>
+'git reflog create' <refs>...
+'git reflog exists' <ref>
 
 Reference logs, or "reflogs", record when the tips of branches and
 other references were updated in the local repository. Reflogs are
diff --git a/builtin/reflog.c b/builtin/reflog.c
index 1ecbfb6..3080865 100644
--- a/builtin/reflog.c
+++ b/builtin/reflog.c
@@ -742,7 +742,7 @@ static int cmd_reflog_create(int argc, const char **argv, const char *prefix)
 
 static int cmd_reflog_exists(int argc, const char **argv, const char *prefix)
 {
-	int i, status = 0, start = 0;
+	int i, start = 0;
 
 	for (i = 1; i < argc; i++) {
 		const char *arg = argv[i];
diff --git a/refs.c b/refs.c
index 55235cf..831d31c 100644
--- a/refs.c
+++ b/refs.c
@@ -2911,7 +2911,7 @@ static int rename_ref_available(const char *oldname, const char *newname)
 }
 
 static int write_ref_to_lockfile(struct ref_lock *lock,
-				 const unsigned char *sha1, struct strbuf* err);
+				 const unsigned char *sha1, struct strbuf *err);
 static int commit_ref_update(struct ref_lock *lock,
 			     const unsigned char *sha1, const char *logmsg,
 			     struct strbuf *err);
-- 
2.5.0-rc0-169-ge0d9173
Previous: David Turner
Message 9 of 9 in “refs backend preamble”
  1. 0/7 refs backend preambleDavid Turner, Jun 29, 2015
  2. 1/7 refs.c: add err arguments to reflog functionsDavid Turner, Jun 29, 2015
  3. 2/7 cherry-pick: treat CHERRY_PICK_HEAD and REVERT_HEAD as refsDavid Turner, Jun 29, 2015
  4. 3/7 bisect: treat BISECT_HEAD as a refDavid Turner, Jun 29, 2015
  5. 4/7 refs: Break out check for reflog autocreationDavid Turner, Jun 29, 2015
  6. 5/7 refs: new public ref function: safe_create_reflogDavid Turner, Jun 29, 2015
  7. 6/7 git-reflog: add create and exists functionsDavid Turner, Jun 29, 2015
  8. 7/7 git-stash: use git-reflog instead of creating filesDavid Turner, Jun 29, 2015
  9. Junio C HamanoJun 29, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.