From: Stefan Beller <sbeller@google.com>
To: git@vger.kernel.org
Cc: jrnieder@gmail.com, gitster@pobox.com, jonathantanmy@google.com,
bmwill@google.com, peff@peff.net, mhagger@alum.mit.edu,
Stefan Beller <sbeller@google.com>
Subject: [PATCHv2 18/20] diff.c: emit_line includes whitespace highlighting
Date: Tue, 16 May 2017 19:58:55 -0700 [thread overview]
Message-ID: <20170517025857.32320-19-sbeller@google.com> (raw)
In-Reply-To: <20170517025857.32320-1-sbeller@google.com>
Currently any whitespace highlighting happens outside the emit_line
function. Teach the highlighting to emit_line, triggered by a new
parameter.
Signed-off-by: Stefan Beller <sbeller@google.com>
---
diff.c | 104 +++++++++++++++++++++++++++++++++++-------------------------
diff.h | 4 ++-
submodule.c | 14 ++++----
3 files changed, 71 insertions(+), 51 deletions(-)
diff --git a/diff.c b/diff.c
index 964b5cb5a7..34482a6a09 100644
--- a/diff.c
+++ b/diff.c
@@ -516,15 +516,33 @@ static void check_blank_at_eof(mmfile_t *mf1, mmfile_t *mf2,
ecbdata->blank_at_eof_in_postimage = (at - l2) + 1;
}
-void emit_line(struct diff_options *o, const char *set, const char *reset,
- int add_line_prefix, int sign, const char *line, int len)
+void emit_line(struct diff_options *o,
+ const char *set, const char *reset,
+ int add_line_prefix, int markup_ws,
+ int sign, const char *line, int len)
{
+ const char *ws;
int has_trailing_newline, has_trailing_carriage_return;
FILE *file = o->file;
if (add_line_prefix)
fputs(diff_line_prefix(o), file);
+ if (markup_ws) {
+ ws = diff_get_color(o->use_color, DIFF_WHITESPACE);
+
+ if (set)
+ fputs(set, file);
+ if (sign)
+ fputc(sign, file);
+ if (reset)
+ fputs(reset, file);
+ ws = diff_get_color(o->use_color, DIFF_WHITESPACE);
+ ws_check_emit(line, len, o->ws_rule,
+ file, set, reset, ws);
+ return;
+ }
+
has_trailing_newline = (len > 0 && line[len-1] == '\n');
if (has_trailing_newline)
len--;
@@ -558,7 +576,7 @@ void emit_line_fmt(struct diff_options *o,
strbuf_vaddf(&sb, fmt, ap);
va_end(ap);
- emit_line(o, set, reset, add_line_prefix, 0, sb.buf, sb.len);
+ emit_line(o, set, reset, add_line_prefix, 0, 0, sb.buf, sb.len);
strbuf_release(&sb);
}
@@ -590,16 +608,15 @@ static void emit_line_checked(const char *reset,
}
if (!ws)
- emit_line(ecbdata->opt, set, reset, 1, sign, line, len);
+ emit_line(ecbdata->opt, set, reset, 1, 0, sign, line, len);
else if (sign == '+' && new_blank_line_at_eof(ecbdata, line, len))
/* Blank line at EOF - paint '+' as well */
- emit_line(ecbdata->opt, ws, reset, 1, sign, line, len);
+ emit_line(ecbdata->opt, ws, reset, 1, 1, sign, line, len);
else {
/* Emit just the prefix, then the rest. */
- emit_line(ecbdata->opt, set, reset, 1, sign, "", 0);
- ws_check_emit(line, len, ecbdata->ws_rule,
- ecbdata->opt->file, set, reset, ws);
+ emit_line(ecbdata->opt, set, reset, 1, 1, sign, line, len);
}
+
}
static void emit_add_line(const char *reset,
@@ -646,7 +663,7 @@ static void emit_hunk_header(struct emit_callback *ecbdata,
if (len < 10 ||
memcmp(line, atat, 2) ||
!(ep = memmem(line + 2, len - 2, atat, 2))) {
- emit_line(ecbdata->opt, context, reset, 1, 0, line, len);
+ emit_line(ecbdata->opt, context, reset, 1, 0, 0, line, len);
return;
}
ep += 2; /* skip over @@ */
@@ -682,7 +699,7 @@ static void emit_hunk_header(struct emit_callback *ecbdata,
strbuf_add(&msgbuf, line + len, org_len - len);
strbuf_complete_line(&msgbuf);
- emit_line(ecbdata->opt, "", "", 1, 0, msgbuf.buf, msgbuf.len);
+ emit_line(ecbdata->opt, "", "", 1, 0, 0, msgbuf.buf, msgbuf.len);
strbuf_release(&msgbuf);
}
@@ -755,7 +772,7 @@ static void emit_rewrite_lines(struct emit_callback *ecb,
static const char *nneof = "\\ No newline at end of file\n";
const char *context = diff_get_color(ecb->color_diff,
DIFF_CONTEXT);
- emit_line(ecb->opt, context, reset, 1, 0,
+ emit_line(ecb->opt, context, reset, 1, 0, 0,
nneof, strlen(nneof));
strbuf_release(&sb);
}
@@ -831,7 +848,7 @@ static void emit_rewrite_diff(const char *name_a,
strbuf_addstr(&out, " +");
add_line_count(&out, lc_b);
strbuf_addstr(&out, " @@\n");
- emit_line(o, fraginfo, reset, 1, 0, out.buf, out.len);
+ emit_line(o, fraginfo, reset, 1, 0, 0, out.buf, out.len);
strbuf_release(&out);
if (lc_a && !o->irreversible_delete)
@@ -904,7 +921,7 @@ static int fn_out_diff_words_write_helper(struct diff_options *o,
while (count) {
char *p = memchr(buf, '\n', count);
if (print)
- emit_line(o, NULL, NULL, 1, 0, "", 0);
+ emit_line(o, NULL, NULL, 1, 0, 0, "", 0);
if (p != buf) {
const char *reset = st_el->color && *st_el->color ?
@@ -913,14 +930,14 @@ static int fn_out_diff_words_write_helper(struct diff_options *o,
strbuf_add(&sb, buf, p ? p - buf : count);
strbuf_addstr(&sb, st_el->suffix);
emit_line(o, st_el->color, reset,
- 0, 0, sb.buf, sb.len);
+ 0, 0, 0, sb.buf, sb.len);
strbuf_reset(&sb);
}
if (!p)
goto out;
strbuf_addstr(&sb, newline);
- emit_line(o, NULL, NULL, 0, 0, sb.buf, sb.len);
+ emit_line(o, NULL, NULL, 0, 0, 0, sb.buf, sb.len);
strbuf_reset(&sb);
count -= p + 1 - buf;
buf = p + 1;
@@ -1135,7 +1152,7 @@ static void diff_words_show(struct diff_words_data *diff_words)
if (diff_words->current_plus != diff_words->plus.text.ptr +
diff_words->plus.text.size) {
if (color_words_output_graph_prefix(diff_words))
- emit_line(diff_words->opt, NULL, NULL, 1, 0, "", 0);
+ emit_line(diff_words->opt, NULL, NULL, 1, 0, 0, "", 0);
fn_out_diff_words_write_helper(diff_words->opt,
&style->ctx, style->newline,
diff_words->plus.text.ptr + diff_words->plus.text.size
@@ -1294,7 +1311,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
o->found_changes = 1;
if (ecbdata->header) {
- emit_line(o, NULL, NULL, 0, 0,
+ emit_line(o, NULL, NULL, 0, 0, 0,
ecbdata->header->buf, ecbdata->header->len);
strbuf_release(ecbdata->header);
ecbdata->header = NULL;
@@ -1347,8 +1364,8 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
}
diff_words_flush(ecbdata);
if (ecbdata->diff_words->type == DIFF_WORDS_PORCELAIN) {
- emit_line(o, context, reset, 1, 0, line, len);
- emit_line(o, NULL, NULL, 0, 0, "~\n", 2);
+ emit_line(o, context, reset, 1, 0, 0, line, len);
+ emit_line(o, NULL, NULL, 0, 0, 0, "~\n", 2);
} else {
/*
* Skip the prefix character, if any. With
@@ -1359,7 +1376,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
line++;
len--;
}
- emit_line(o, context, reset, 1, 0, line, len);
+ emit_line(o, context, reset, 1, 0, 0, line, len);
}
return;
}
@@ -1382,7 +1399,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
/* incomplete line at the end */
ecbdata->lno_in_preimage++;
emit_line(o, diff_get_color(ecbdata->color_diff, DIFF_CONTEXT),
- reset, 1, 0, line, len);
+ reset, 1, 0, 0, line, len);
break;
}
}
@@ -1571,7 +1588,7 @@ void print_stat_summary_0(struct diff_options *options, int files,
if (!files) {
assert(insertions == 0 && deletions == 0);
strbuf_addstr(&sb, " 0 files changed");
- emit_line(options, NULL, NULL, 1, 0, sb.buf, sb.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, sb.buf, sb.len);
return;
}
@@ -1599,7 +1616,7 @@ void print_stat_summary_0(struct diff_options *options, int files,
deletions);
}
strbuf_addch(&sb, '\n');
- emit_line(options, NULL, NULL, 1, 0, sb.buf, sb.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, sb.buf, sb.len);
strbuf_release(&sb);
}
@@ -1783,7 +1800,7 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
strbuf_addf(&out, " %*s", number_width, "Bin");
if (!added && !deleted) {
strbuf_addch(&out, '\n');
- emit_line(options, NULL, NULL, 1, 0, out.buf, out.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, out.buf, out.len);
strbuf_reset(&out);
continue;
}
@@ -1793,14 +1810,14 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
strbuf_addf(&out, "%s%"PRIuMAX"%s",
add_c, added, reset);
strbuf_addstr(&out, " bytes\n");
- emit_line(options, NULL, NULL, 1, 0, out.buf, out.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, out.buf, out.len);
strbuf_reset(&out);
continue;
}
else if (file->is_unmerged) {
show_name(&out, prefix, name, len);
strbuf_addstr(&out, " Unmerged\n");
- emit_line(options, NULL, NULL, 1, 0, out.buf, out.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, out.buf, out.len);
strbuf_reset(&out);
continue;
}
@@ -1831,7 +1848,7 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
show_graph(&out, '+', add, add_c, reset);
show_graph(&out, '-', del, del_c, reset);
strbuf_addch(&out, '\n');
- emit_line(options, NULL, NULL, 1, 0, out.buf, out.len);
+ emit_line(options, NULL, NULL, 1, 0, 0, out.buf, out.len);
strbuf_reset(&out);
}
@@ -1853,7 +1870,7 @@ static void show_stats(struct diffstat_t *data, struct diff_options *options)
if (i < count)
continue;
if (!extra_shown)
- emit_line(options, NULL, NULL, 1, 0,
+ emit_line(options, NULL, NULL, 1, 0, 0,
" ...\n", strlen(" ...\n"));
extra_shown = 1;
}
@@ -2207,7 +2224,7 @@ static void checkdiff_consume(void *priv, char *line, unsigned long len)
fprintf(data->o->file, "%s%s:%d: %s.\n",
line_prefix, data->filename, data->lineno, err);
free(err);
- emit_line(data->o, set, reset, 1, 0, line, 1);
+ emit_line(data->o, set, reset, 1, 0, 0, line, 1);
ws_check_emit(line + 1, len - 1, data->ws_rule,
data->o->file, set, reset, ws);
} else if (line[0] == ' ') {
@@ -2303,9 +2320,9 @@ static void emit_binary_diff_body(struct diff_options *o,
line[len++] = '\n';
line[len] = '\0';
- emit_line(o, NULL, NULL, 1, 0, line, len);
+ emit_line(o, NULL, NULL, 1, 0, 0, line, len);
}
- emit_line(o, NULL, NULL, 1, 0, "\n", 1);
+ emit_line(o, NULL, NULL, 1, 0, 0, "\n", 1);
free(data);
}
@@ -2314,7 +2331,7 @@ static void emit_binary_diff(struct diff_options *o,
{
const char *s = "GIT binary patch\n";
const int len = strlen(s);
- emit_line(o, NULL, NULL, 1, 0, s, len);
+ emit_line(o, NULL, NULL, 1, 0, 0, s, len);
emit_binary_diff_body(o, one, two);
emit_binary_diff_body(o, two, one);
}
@@ -2457,7 +2474,7 @@ static void builtin_diff(const char *name_a,
if (complete_rewrite &&
(textconv_one || !diff_filespec_is_binary(one)) &&
(textconv_two || !diff_filespec_is_binary(two))) {
- emit_line(o, NULL, NULL, 0, 0, header.buf, header.len);
+ emit_line(o, NULL, NULL, 0, 0, 0, header.buf, header.len);
strbuf_reset(&header);
emit_rewrite_diff(name_a, name_b, one, two,
textconv_one, textconv_two, o);
@@ -2467,7 +2484,7 @@ static void builtin_diff(const char *name_a,
}
if (o->irreversible_delete && lbl[1][0] == '/') {
- emit_line(o, NULL, NULL, 0, 0, header.buf, header.len);
+ emit_line(o, NULL, NULL, 0, 0, 0, header.buf, header.len);
strbuf_reset(&header);
goto free_ab_and_return;
} else if (!DIFF_OPT_TST(o, TEXT) &&
@@ -2478,11 +2495,11 @@ static void builtin_diff(const char *name_a,
!DIFF_OPT_TST(o, BINARY)) {
if (!oidcmp(&one->oid, &two->oid)) {
if (must_show_header)
- emit_line(o, NULL, NULL, 0, 0,
+ emit_line(o, NULL, NULL, 0, 0, 0,
header.buf, header.len);
goto free_ab_and_return;
}
- emit_line(o, NULL, NULL, 0, 0,
+ emit_line(o, NULL, NULL, 0, 0, 0,
header.buf, header.len);
emit_line_fmt(o, NULL, NULL, 1,
"Binary files %s and %s differ\n",
@@ -2495,11 +2512,11 @@ static void builtin_diff(const char *name_a,
if (mf1.size == mf2.size &&
!memcmp(mf1.ptr, mf2.ptr, mf1.size)) {
if (must_show_header)
- emit_line(o, NULL, NULL, 0, 0,
+ emit_line(o, NULL, NULL, 0, 0, 0,
header.buf, header.len);
goto free_ab_and_return;
}
- emit_line(o, NULL, NULL, 0, 0,
+ emit_line(o, NULL, NULL, 0, 0, 0,
header.buf, header.len);
strbuf_reset(&header);
if (DIFF_OPT_TST(o, BINARY))
@@ -2519,7 +2536,7 @@ static void builtin_diff(const char *name_a,
const struct userdiff_funcname *pe;
if (must_show_header) {
- emit_line(o, NULL, NULL, 0, 0, header.buf, header.len);
+ emit_line(o, NULL, NULL, 0, 0, 0, header.buf, header.len);
strbuf_reset(&header);
}
@@ -2536,6 +2553,7 @@ static void builtin_diff(const char *name_a,
ecbdata.label_path = lbl;
ecbdata.color_diff = want_color(o->use_color);
ecbdata.ws_rule = whitespace_rule(name_b);
+ o->ws_rule = ecbdata.ws_rule;
if (ecbdata.ws_rule & WS_BLANK_AT_EOF)
check_blank_at_eof(&mf1, &mf2, &ecbdata);
ecbdata.opt = o;
@@ -4510,7 +4528,7 @@ static void show_file_mode_name(struct diff_options *opt, const char *newdelete,
quote_c_style(fs->path, &sb, NULL, 0);
strbuf_addch(&sb, '\n');
- emit_line(opt, NULL, NULL, 1, 0, sb.buf, sb.len);
+ emit_line(opt, NULL, NULL, 1, 0, 0, sb.buf, sb.len);
strbuf_release(&sb);
}
@@ -4562,7 +4580,7 @@ static void diff_summary(struct diff_options *opt, struct diff_filepair *p)
strbuf_addstr(&sb, " rewrite ");
quote_c_style(p->two->path, &sb, NULL, 0);
strbuf_addf(&sb, " (%d%%)\n", similarity_index(p));
- emit_line(opt, NULL, NULL, 1, 0, sb.buf, sb.len);
+ emit_line(opt, NULL, NULL, 1, 0, 0, sb.buf, sb.len);
}
show_mode_change(opt, p, !p->score);
break;
@@ -4869,10 +4887,10 @@ void diff_flush(struct diff_options *options)
term[0] = options->line_termination;
term[1] = '\0';
- emit_line(options, NULL, NULL, 1, 0, term, !!term[0]);
+ emit_line(options, NULL, NULL, 1, 0, 0, term, !!term[0]);
if (options->stat_sep) {
/* attach patch instead of inline */
- emit_line(options, NULL, NULL, 0, 0,
+ emit_line(options, NULL, NULL, 0, 0, 0,
options->stat_sep,
strlen(options->stat_sep));
}
diff --git a/diff.h b/diff.h
index b75b0d7283..267acf1980 100644
--- a/diff.h
+++ b/diff.h
@@ -186,12 +186,14 @@ struct diff_options {
void *output_prefix_data;
int diff_path_counter;
+
+ unsigned ws_rule;
};
void emit_line_fmt(struct diff_options *o, const char *set, const char *reset,
int add_line_prefix, const char *fmt, ...);
void emit_line(struct diff_options *o, const char *set, const char *reset,
- int add_line_prefix, int sign, const char *line, int len);
+ int add_line_prefix, int markup_ws, int sign, const char *line, int len);
enum color_diff {
DIFF_RESET = 0,
diff --git a/submodule.c b/submodule.c
index 5996ebca44..868f913971 100644
--- a/submodule.c
+++ b/submodule.c
@@ -378,9 +378,9 @@ static void print_submodule_summary(struct rev_info *rev,
format_commit_message(commit, format, &sb, &ctx);
strbuf_addch(&sb, '\n');
if (commit->object.flags & SYMMETRIC_LEFT)
- emit_line(o, del, reset, 1, 0, sb.buf, sb.len);
+ emit_line(o, del, reset, 1, 0, 0, sb.buf, sb.len);
else if (add)
- emit_line(o, add, reset, 1, 0, sb.buf, sb.len);
+ emit_line(o, add, reset, 1, 0, 0, sb.buf, sb.len);
}
strbuf_release(&sb);
}
@@ -473,7 +473,7 @@ static void show_submodule_header(struct diff_options *o, const char *path,
strbuf_addf(&sb, " %s\n", message);
else
strbuf_addf(&sb, "%s:\n", fast_backward ? " (rewind)" : "");
- emit_line(o, meta, reset, 1, 0, sb.buf, sb.len);
+ emit_line(o, meta, reset, 1, 0, 0, sb.buf, sb.len);
strbuf_release(&sb);
}
@@ -501,7 +501,7 @@ void show_submodule_summary(struct diff_options *o, const char *path,
/* Treat revision walker failure the same as missing commits */
if (prepare_submodule_summary(&rev, path, left, right, merge_bases)) {
const char *error = "(revision walker failed)\n";
- emit_line(o, NULL, NULL, 1, 0, error, strlen(error));
+ emit_line(o, NULL, NULL, 1, 0, 0, error, strlen(error));
goto out;
}
@@ -570,15 +570,15 @@ void show_submodule_inline_diff(struct diff_options *o, const char *path,
prepare_submodule_repo_env(&cp.env_array);
if (start_command(&cp)) {
const char *error = "(diff failed)\n";
- emit_line(o, NULL, NULL, 1, 0, error, strlen(error));
+ emit_line(o, NULL, NULL, 1, 0, 0, error, strlen(error));
}
while (strbuf_getwholeline_fd(&sb, cp.out, '\n') != EOF)
- emit_line(o, NULL, NULL, 1, 0, sb.buf, sb.len);
+ emit_line(o, NULL, NULL, 1, 0, 0, sb.buf, sb.len);
if (finish_command(&cp)) {
const char *error = "(diff failed)\n";
- emit_line(o, NULL, NULL, 1, 0, error, strlen(error));
+ emit_line(o, NULL, NULL, 1, 0, 0, error, strlen(error));
}
done:
--
2.13.0.18.g7d86cc8ba0
next prev parent reply other threads:[~2017-05-17 2:59 UTC|newest]
Thread overview: 128+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-14 4:00 [RFC PATCH 00/19] Diff machine: highlight moved lines Stefan Beller
2017-05-14 4:00 ` [PATCH 01/19] diff: readability fix Stefan Beller
2017-05-14 4:01 ` [PATCH 02/19] diff: move line ending check into emit_hunk_header Stefan Beller
2017-05-15 6:48 ` Junio C Hamano
2017-05-15 16:13 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 03/19] diff.c: drop 'nofirst' from emit_line_0 Stefan Beller
2017-05-15 18:26 ` Jonathan Tan
2017-05-15 18:33 ` Stefan Beller
2017-05-16 16:05 ` Jonathan Tan
2017-05-15 19:22 ` Brandon Williams
2017-05-15 19:35 ` Stefan Beller
2017-05-15 19:45 ` Brandon Williams
2017-05-14 4:01 ` [PATCH 04/19] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-05-14 4:01 ` [PATCH 05/19] diff.c: emit_line_0 can handle no color setting Stefan Beller
2017-05-15 18:31 ` Jonathan Tan
2017-05-15 22:11 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 06/19] diff: add emit_line_fmt Stefan Beller
2017-05-15 19:31 ` Brandon Williams
2017-05-14 4:01 ` [PATCH 07/19] diff.c: convert fn_out_consume to use emit_line_* Stefan Beller
2017-05-16 1:00 ` Junio C Hamano
2017-05-16 1:05 ` Junio C Hamano
2017-05-16 16:23 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 08/19] diff.c: convert builtin_diff " Stefan Beller
2017-05-15 18:42 ` Jonathan Tan
2017-05-14 4:01 ` [PATCH 09/19] diff.c: convert emit_rewrite_diff " Stefan Beller
2017-05-14 4:01 ` [PATCH 10/19] diff.c: convert emit_rewrite_lines " Stefan Beller
2017-05-15 19:09 ` Jonathan Tan
2017-05-15 19:31 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 11/19] submodule.c: convert show_submodule_summary to use emit_line_fmt Stefan Beller
2017-05-14 4:01 ` [PATCH 12/19] diff.c: convert emit_binary_diff_body to use emit_line_* Stefan Beller
2017-05-14 4:01 ` [PATCH 13/19] diff.c: convert show_stats " Stefan Beller
2017-05-14 4:01 ` [PATCH 14/19] diff.c: convert word diffing " Stefan Beller
2017-05-15 22:40 ` Jonathan Tan
2017-05-15 23:12 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 15/19] diff.c: convert diff_flush " Stefan Beller
2017-05-15 20:21 ` Jonathan Tan
2017-05-15 22:08 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 16/19] diff.c: convert diff_summary " Stefan Beller
2017-05-14 4:01 ` [PATCH 17/19] diff.c: factor out emit_line_ws for coloring whitespaces Stefan Beller
2017-05-14 4:01 ` [PATCH 18/19] diff: buffer all output if asked to Stefan Beller
2017-05-14 4:06 ` Jeff King
2017-05-14 4:25 ` Stefan Beller
2017-05-16 4:14 ` Jonathan Tan
2017-05-16 16:42 ` Stefan Beller
2017-05-14 4:01 ` [PATCH 19/19] diff.c: color moved lines differently Stefan Beller
2017-05-15 22:42 ` Brandon Williams
2017-05-16 4:34 ` Jonathan Tan
2017-05-16 12:31 ` Jeff King
2017-05-15 12:43 ` [RFC PATCH 00/19] Diff machine: highlight moved lines Junio C Hamano
2017-05-15 16:33 ` Stefan Beller
2017-05-17 2:58 ` [PATCHv2 00/20] " Stefan Beller
2017-05-17 2:58 ` [PATCHv2 01/20] diff: readability fix Stefan Beller
2017-05-17 2:58 ` [PATCHv2 02/20] diff: move line ending check into emit_hunk_header Stefan Beller
2017-05-17 2:58 ` [PATCHv2 03/20] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-05-17 2:58 ` [PATCHv2 04/20] diff.c: teach emit_line_0 to accept sign parameter Stefan Beller
2017-05-17 2:58 ` [PATCHv2 05/20] diff.c: emit_line_0 can handle no color setting Stefan Beller
2017-05-17 2:58 ` [PATCHv2 06/20] diff.c: emit_line_0 takes parameter whether to output line prefix Stefan Beller
2017-05-17 2:58 ` [PATCHv2 07/20] diff.c: inline emit_line_0 into emit_line Stefan Beller
2017-05-17 2:58 ` [PATCHv2 08/20] diff.c: convert fn_out_consume to use emit_line Stefan Beller
2017-05-17 2:58 ` [PATCHv2 09/20] diff.c: convert builtin_diff to use emit_line_* Stefan Beller
2017-05-17 2:58 ` [PATCHv2 10/20] diff.c: convert emit_rewrite_diff " Stefan Beller
2017-05-17 2:58 ` [PATCHv2 11/20] diff.c: convert emit_rewrite_lines " Stefan Beller
2017-05-17 5:03 ` Junio C Hamano
2017-05-17 21:16 ` Stefan Beller
2017-05-18 3:35 ` Junio C Hamano
2017-05-17 2:58 ` [PATCHv2 12/20] submodule.c: convert show_submodule_summary to use emit_line_fmt Stefan Beller
2017-05-17 5:19 ` Junio C Hamano
2017-05-17 21:05 ` Stefan Beller
2017-05-18 3:25 ` Junio C Hamano
2017-05-18 17:12 ` Stefan Beller
2017-05-20 4:50 ` Junio C Hamano
2017-05-20 22:00 ` Stefan Beller
2017-05-17 2:58 ` [PATCHv2 13/20] diff.c: convert emit_binary_diff_body to use emit_line_* Stefan Beller
2017-05-17 2:58 ` [PATCHv2 14/20] diff.c: convert show_stats " Stefan Beller
2017-05-17 2:58 ` [PATCHv2 15/20] diff.c: convert word diffing " Stefan Beller
2017-05-17 2:58 ` [PATCHv2 16/20] diff.c: convert diff_flush " Stefan Beller
2017-05-17 2:58 ` [PATCHv2 17/20] diff.c: convert diff_summary " Stefan Beller
2017-05-17 2:58 ` Stefan Beller [this message]
2017-05-17 2:58 ` [PATCHv2 19/20] diff: buffer all output if asked to Stefan Beller
2017-05-17 2:58 ` [PATCHv2 20/20] diff.c: color moved lines differently Stefan Beller
2017-05-18 19:37 ` [PATCHv3 00/20] Diff machine: highlight moved lines Stefan Beller
2017-05-18 19:37 ` [PATCHv3 01/20] diff: readability fix Stefan Beller
2017-05-18 19:37 ` [PATCHv3 02/20] diff: move line ending check into emit_hunk_header Stefan Beller
2017-05-18 19:37 ` [PATCHv3 03/20] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-05-18 19:37 ` [PATCHv3 04/20] diff.c: teach emit_line_0 to accept sign parameter Stefan Beller
2017-05-18 23:33 ` Jonathan Tan
2017-05-22 23:36 ` Stefan Beller
2017-05-18 19:37 ` [PATCHv3 05/20] diff.c: emit_line_0 can handle no color setting Stefan Beller
2017-05-18 19:37 ` [PATCHv3 06/20] diff.c: emit_line_0 takes parameter whether to output line prefix Stefan Beller
2017-05-18 19:37 ` [PATCHv3 07/20] diff.c: inline emit_line_0 into emit_line Stefan Beller
2017-05-18 19:37 ` [PATCHv3 08/20] diff.c: convert fn_out_consume to use emit_line Stefan Beller
2017-05-18 19:37 ` [PATCHv3 09/20] diff.c: convert builtin_diff to use emit_line_* Stefan Beller
2017-05-18 19:37 ` [PATCHv3 10/20] diff.c: convert emit_rewrite_diff " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 11/20] diff.c: convert emit_rewrite_lines " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 12/20] submodule.c: convert show_submodule_summary to use emit_line_fmt Stefan Beller
2017-05-18 19:37 ` [PATCHv3 13/20] diff.c: convert emit_binary_diff_body to use emit_line_* Stefan Beller
2017-05-18 19:37 ` [PATCHv3 14/20] diff.c: convert show_stats " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 15/20] diff.c: convert word diffing " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 16/20] diff.c: convert diff_flush " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 17/20] diff.c: convert diff_summary " Stefan Beller
2017-05-18 19:37 ` [PATCHv3 18/20] diff.c: emit_line includes whitespace highlighting Stefan Beller
2017-05-18 19:37 ` [PATCHv3 19/20] diff: buffer all output if asked to Stefan Beller
2017-05-18 19:37 ` [PATCHv3 20/20] diff.c: color moved lines differently Stefan Beller
2017-05-19 18:23 ` Jonathan Tan
2017-05-19 18:40 ` Stefan Beller
2017-05-19 19:34 ` Jonathan Tan
2017-05-23 2:40 ` [PATCHv4 00/17] Diff machine: highlight moved lines Stefan Beller
2017-05-23 2:40 ` [PATCHv4 01/17] diff: readability fix Stefan Beller
2017-05-23 2:40 ` [PATCHv4 02/17] diff: move line ending check into emit_hunk_header Stefan Beller
2017-05-23 2:40 ` [PATCHv4 03/17] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-05-23 2:40 ` [PATCHv4 04/17] diff: introduce more flexible emit function Stefan Beller
2017-05-23 2:40 ` [PATCHv4 05/17] diff.c: convert fn_out_consume to use emit_line Stefan Beller
2017-05-23 2:40 ` [PATCHv4 06/17] diff.c: convert builtin_diff to use emit_line_* Stefan Beller
2017-05-23 2:40 ` [PATCHv4 07/17] diff.c: convert emit_rewrite_diff " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 08/17] diff.c: convert emit_rewrite_lines " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 09/17] submodule.c: convert show_submodule_summary to use emit_line_fmt Stefan Beller
2017-05-23 5:59 ` Junio C Hamano
2017-05-23 18:14 ` Stefan Beller
2017-05-23 2:40 ` [PATCHv4 10/17] diff.c: convert emit_binary_diff_body to use emit_line_* Stefan Beller
2017-05-23 2:40 ` [PATCHv4 11/17] diff.c: convert show_stats " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 12/17] diff.c: convert word diffing " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 13/17] diff.c: convert diff_flush " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 14/17] diff.c: convert diff_summary " Stefan Beller
2017-05-23 2:40 ` [PATCHv4 15/17] diff.c: emit_line includes whitespace highlighting Stefan Beller
2017-05-23 2:40 ` [PATCHv4 16/17] diff: buffer all output if asked to Stefan Beller
2017-05-23 2:40 ` [PATCHv4 17/17] diff.c: color moved lines differently Stefan Beller
2017-05-27 1:04 ` [PATCHv4 00/17] Diff machine: highlight moved lines Jacob Keller
2017-05-30 21:38 ` Stefan Beller
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170517025857.32320-19-sbeller@google.com \
--to=sbeller@google.com \
--cc=bmwill@google.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jonathantanmy@google.com \
--cc=jrnieder@gmail.com \
--cc=mhagger@alum.mit.edu \
--cc=peff@peff.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).