Skip to content

Commit

Permalink
Sync with 1.8.5.2
Browse files Browse the repository at this point in the history
* maint:
  Git 1.8.5.2
  cmd_repack(): remove redundant local variable "nr_packs"
  • Loading branch information
Junio C Hamano committed Dec 17, 2013
2 parents b10cd57 + 5512ac5 commit 7794a68
Show file tree
Hide file tree
Showing 3 changed files with 24 additions and 5 deletions.
20 changes: 20 additions & 0 deletions Documentation/RelNotes/1.8.5.2.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
Git v1.8.5.2 Release Notes
==========================

Fixes since v1.8.5.1
--------------------

* "git diff -- ':(icase)makefile'" was unnecessarily rejected at the
command line parser.

* "git cat-file --batch-check=ok" did not check the existence of
the named object.

* "git am --abort" sometimes complained about not being able to write
a tree with an 0{40} object in it.

* Two processes creating loose objects at the same time could have
failed unnecessarily when the name of their new objects started
with the same byte value, due to a race condition.

Also contains typofixes, documentation updates and trivial code clean-ups.
3 changes: 2 additions & 1 deletion Documentation/git.txt
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,10 @@ unreleased) version of Git, that is available from 'master'
branch of the `git.git` repository.
Documentation for older releases are available here:

* link:v1.8.5.1/git.html[documentation for release 1.8.5.1]
* link:v1.8.5.2/git.html[documentation for release 1.8.5.2]

* release notes for
link:RelNotes/1.8.5.2.txt[1.8.5.2],
link:RelNotes/1.8.5.1.txt[1.8.5.1],
link:RelNotes/1.8.5.txt[1.8.5].

Expand Down
6 changes: 2 additions & 4 deletions builtin/repack.c
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ int cmd_repack(int argc, const char **argv, const char *prefix)
struct string_list rollback = STRING_LIST_INIT_NODUP;
struct string_list existing_packs = STRING_LIST_INIT_DUP;
struct strbuf line = STRBUF_INIT;
int nr_packs, ext, ret, failed;
int ext, ret, failed;
FILE *out;

/* variables to be filled by option parsing */
Expand Down Expand Up @@ -233,21 +233,19 @@ int cmd_repack(int argc, const char **argv, const char *prefix)
if (ret)
return ret;

nr_packs = 0;
out = xfdopen(cmd.out, "r");
while (strbuf_getline(&line, out, '\n') != EOF) {
if (line.len != 40)
die("repack: Expecting 40 character sha1 lines only from pack-objects.");
string_list_append(&names, line.buf);
nr_packs++;
}
fclose(out);
ret = finish_command(&cmd);
if (ret)
return ret;
argv_array_clear(&cmd_args);

if (!nr_packs && !quiet)
if (!names.nr && !quiet)
printf("Nothing new to pack.\n");

/*
Expand Down

0 comments on commit 7794a68

Please sign in to comment.