Skip to content

Commit

Permalink
builtin-commit: Two trivial style-cleanups
Browse files Browse the repository at this point in the history
Pierre Habouzit noticed that two variables were not static which should
have been, and that adding "\n\n" is better than adding '\n' twice.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
Johannes Schindelin authored and Junio C Hamano committed Jul 23, 2008
1 parent ea3d988 commit 3b6aeb3
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions builtin-commit.c
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ static enum {
static char *cleanup_arg;

static int use_editor = 1, initial_commit, in_merge;
const char *only_include_assumed;
struct strbuf message;
static const char *only_include_assumed;
static struct strbuf message;

static int opt_parse_m(const struct option *opt, const char *arg, int unset)
{
Expand All @@ -78,8 +78,7 @@ static int opt_parse_m(const struct option *opt, const char *arg, int unset)
strbuf_setlen(buf, 0);
else {
strbuf_addstr(buf, arg);
strbuf_addch(buf, '\n');
strbuf_addch(buf, '\n');
strbuf_addstr(buf, "\n\n");
}
return 0;
}
Expand Down

0 comments on commit 3b6aeb3

Please sign in to comment.