mirror of
https://github.com/git/git
synced 2024-08-28 20:20:07 +00:00
branch: add edit_description() helper
Rather than have a variable with a short name that is fed to git_path(), let's add a helper function that returns the full path. This avoids the dangerous git_path() function. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
f5d284f6df
commit
c10388c7dc
|
@ -600,7 +600,7 @@ static void rename_branch(const char *oldname, const char *newname, int force)
|
|||
strbuf_release(&newsection);
|
||||
}
|
||||
|
||||
static const char edit_description[] = "BRANCH_DESCRIPTION";
|
||||
static GIT_PATH_FUNC(edit_description, "EDIT_DESCRIPTION")
|
||||
|
||||
static int edit_branch_description(const char *branch_name)
|
||||
{
|
||||
|
@ -615,9 +615,9 @@ static int edit_branch_description(const char *branch_name)
|
|||
" %s\n"
|
||||
"Lines starting with '%c' will be stripped.\n"),
|
||||
branch_name, comment_line_char);
|
||||
write_file_buf(git_path(edit_description), buf.buf, buf.len);
|
||||
write_file_buf(edit_description(), buf.buf, buf.len);
|
||||
strbuf_reset(&buf);
|
||||
if (launch_editor(git_path(edit_description), &buf, NULL)) {
|
||||
if (launch_editor(edit_description(), &buf, NULL)) {
|
||||
strbuf_release(&buf);
|
||||
return -1;
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue