Skip to content
This repository was archived by the owner on Nov 9, 2017. It is now read-only.

Commit ce37596

Browse files
committed
builtin/log.c: drop redundant "numbered_files" parameter from make_cover_letter()
Signed-off-by: Junio C Hamano <[email protected]>
1 parent 25a751f commit ce37596

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

builtin/log.c

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -768,7 +768,6 @@ static void add_branch_description(struct strbuf *buf, const char *branch_name)
768768
}
769769

770770
static void make_cover_letter(struct rev_info *rev, int use_stdout,
771-
int numbered_files,
772771
struct commit *origin,
773772
int nr, struct commit **list, struct commit *head,
774773
const char *branch_name,
@@ -791,7 +790,7 @@ static void make_cover_letter(struct rev_info *rev, int use_stdout,
791790
committer = git_committer_info(0);
792791

793792
if (!use_stdout &&
794-
reopen_stdout(NULL, numbered_files ? NULL : "cover-letter", rev, quiet))
793+
reopen_stdout(NULL, rev->numbered_files ? NULL : "cover-letter", rev, quiet))
795794
return;
796795

797796
log_write_email_headers(rev, head, &pp.subject, &pp.after_subject,
@@ -1045,7 +1044,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
10451044
int nr = 0, total, i;
10461045
int use_stdout = 0;
10471046
int start_number = -1;
1048-
int numbered_files = 0; /* _just_ numbers */
1047+
int just_numbers = 0;
10491048
int ignore_if_in_upstream = 0;
10501049
int cover_letter = 0;
10511050
int boundary_count = 0;
@@ -1070,7 +1069,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
10701069
N_("print patches to standard out")),
10711070
OPT_BOOLEAN(0, "cover-letter", &cover_letter,
10721071
N_("generate a cover letter")),
1073-
OPT_BOOLEAN(0, "numbered-files", &numbered_files,
1072+
OPT_BOOLEAN(0, "numbered-files", &just_numbers,
10741073
N_("use simple number sequence for output file names")),
10751074
OPT_STRING(0, "suffix", &fmt_patch_suffix, N_("sfx"),
10761075
N_("use <sfx> instead of '.patch'")),
@@ -1338,12 +1337,12 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
13381337
const char *msgid = clean_message_id(in_reply_to);
13391338
string_list_append(rev.ref_message_ids, msgid);
13401339
}
1341-
rev.numbered_files = numbered_files;
1340+
rev.numbered_files = just_numbers;
13421341
rev.patch_suffix = fmt_patch_suffix;
13431342
if (cover_letter) {
13441343
if (thread)
13451344
gen_message_id(&rev, "cover");
1346-
make_cover_letter(&rev, use_stdout, numbered_files,
1345+
make_cover_letter(&rev, use_stdout,
13471346
origin, nr, list, head, branch_name, quiet);
13481347
total++;
13491348
start_number--;
@@ -1390,7 +1389,7 @@ int cmd_format_patch(int argc, const char **argv, const char *prefix)
13901389
}
13911390

13921391
if (!use_stdout &&
1393-
reopen_stdout(numbered_files ? NULL : commit, NULL, &rev, quiet))
1392+
reopen_stdout(rev.numbered_files ? NULL : commit, NULL, &rev, quiet))
13941393
die(_("Failed to create output files"));
13951394
shown = log_tree_commit(&rev, commit);
13961395
free(commit->buffer);

0 commit comments

Comments
 (0)