summaryrefslogtreecommitdiff
path: root/builtin-blame.c
diff options
context:
space:
mode:
authorJunio C Hamano <junkio@cox.net>2007-01-28 01:42:31 -0800
committerJunio C Hamano <junkio@cox.net>2007-01-28 02:04:48 -0800
commit46e5e69d5f2e8ae87a29953531a3d1bd7cb0292f (patch)
tree878bf8410b8c1142c37b2a8133f73330c55ed8e7 /builtin-blame.c
parent717d1462ba0ebc430775eccaae65eb9b8234eb32 (diff)
downloadgit-46e5e69d5f2e8ae87a29953531a3d1bd7cb0292f.tar.gz
git-blame --porcelain: quote filename in c-style when needed.
Otherwise a pathname that has funny characters such as LF would screw up the parsing programs of the output. Strictly speaking, this is not backward compatible, but the current output for pathnames that have embedded LF and such cannot be sanely parsed anyway, and pathnames that only use characters from the portable pathname character set won't be affected. Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'builtin-blame.c')
-rw-r--r--builtin-blame.c15
1 files changed, 10 insertions, 5 deletions
diff --git a/builtin-blame.c b/builtin-blame.c
index 54ab675624..7a58ee303f 100644
--- a/builtin-blame.c
+++ b/builtin-blame.c
@@ -1182,6 +1182,13 @@ static void get_commit_info(struct commit *commit,
summary_buf[len] = 0;
}
+static void write_filename_info(const char *path)
+{
+ printf("filename ");
+ write_name_quoted(NULL, 0, path, 1, stdout);
+ putchar('\n');
+}
+
static void found_guilty_entry(struct blame_entry *ent)
{
if (ent->guilty)
@@ -1209,9 +1216,7 @@ static void found_guilty_entry(struct blame_entry *ent)
if (suspect->commit->object.flags & UNINTERESTING)
printf("boundary\n");
}
- printf("filename ");
- write_name_quoted(NULL, 0, suspect->path, 1, stdout);
- putchar('\n');
+ write_filename_info(suspect->path);
}
}
@@ -1315,13 +1320,13 @@ static void emit_porcelain(struct scoreboard *sb, struct blame_entry *ent)
printf("committer-mail %s\n", ci.committer_mail);
printf("committer-time %lu\n", ci.committer_time);
printf("committer-tz %s\n", ci.committer_tz);
- printf("filename %s\n", suspect->path);
+ write_filename_info(suspect->path);
printf("summary %s\n", ci.summary);
if (suspect->commit->object.flags & UNINTERESTING)
printf("boundary\n");
}
else if (suspect->commit->object.flags & MORE_THAN_ONE_PATH)
- printf("filename %s\n", suspect->path);
+ write_filename_info(suspect->path);
cp = nth_line(sb, ent->lno);
for (cnt = 0; cnt < ent->num_lines; cnt++) {