git@vger.kernel.org mailing list mirror (one of many)
 help / Atom feed
From: "Philip Oakley" <philipoakley@iee.org>
To: "Stefan Beller" <sbeller@google.com>, <gitster@pobox.com>
Cc: <git@vger.kernel.org>, <bmwill@google.com>, <jrnieder@gmail.com>, <peff@peff.net>, <mhagger@alum.mit.edu>, <jonathantanmy@google.com>, "Stefan Beller" <sbeller@google.com>
Subject: Re: [PATCH 1/1] diff.c: color moved lines differently
Date: Sat, 27 May 2017 08:05:48 +0100
Message-ID: <39D67E58AC02490FB6D6124BDF3EE2A9@PhilipOakley> (raw)
In-Reply-To: <20170527001820.25214-2-sbeller@google.com>

a couple of mispellings in the doc parts:
  s/on location/one location/
[code not checked]
----- Original Message ----- 
From: "Stefan Beller" <sbeller@google.com>
Subject: [PATCH 1/1] diff.c: color moved lines differently


> When a patch consists mostly of moving blocks of code around, it can
> be quite tedious to ensure that the blocks are moved verbatim, and not
> undesirably modified in the move. To that end, color blocks that are
> moved within the same patch differently. For example (OM, del, add,
> and NM are different colors):
>
>    [OM]  -void sensitive_stuff(void)
>    [OM]  -{
>    [OM]  -        if (!is_authorized_user())
>    [OM]  -                die("unauthorized");
>    [OM]  -        sensitive_stuff(spanning,
>    [OM]  -                        multiple,
>    [OM]  -                        lines);
>    [OM]  -}
>
>           void another_function()
>           {
>    [del] -        printf("foo");
>    [add] +        printf("bar");
>           }
>
>    [NM]  +void sensitive_stuff(void)
>    [NM]  +{
>    [NM]  +        if (!is_authorized_user())
>    [NM]  +                die("unauthorized");
>    [NM]  +        sensitive_stuff(spanning,
>    [NM]  +                        multiple,
>    [NM]  +                        lines);
>    [NM]  +}
>
> However adjacent blocks may be problematic. For example, in this
> potentially malicious patch, the swapping of blocks can be spotted:
>
>    [OM]  -void sensitive_stuff(void)
>    [OM]  -{
>    [OMA] -        if (!is_authorized_user())
>    [OMA] -                die("unauthorized");
>    [OM]  -        sensitive_stuff(spanning,
>    [OM]  -                        multiple,
>    [OM]  -                        lines);
>    [OMA] -}
>
>           void another_function()
>           {
>    [del] -        printf("foo");
>    [add] +        printf("bar");
>           }
>
>    [NM]  +void sensitive_stuff(void)
>    [NM]  +{
>    [NMA] +        sensitive_stuff(spanning,
>    [NMA] +                        multiple,
>    [NMA] +                        lines);
>    [NM]  +        if (!is_authorized_user())
>    [NM]  +                die("unauthorized");
>    [NMA] +}
>
> If the moved code is larger, it is easier to hide some permutation in the
> code, which is why some alternative coloring is needed.
>
> As the reviewers attention should be brought to the places, where the
> difference is introduced to the moved code, we cannot just have one new
> color for all of moved code.
>
> First I implemented an alternative design, which would try to fingerprint
> a line by its neighbors to detect if we are in a block or at the boundary.
> This idea iss error prone as it inspected each line and its neighboring
> lines to determine if the line was (a) moved and (b) if was deep inside
> a hunk by having matching neighboring lines. This is unreliable as the
> we can construct hunks which have equal neighbors that just exceed the
> number of lines inspected. (Think of 'AXYZBXYZCXYZD..' with each letter
> as a line, that is permutated to AXYZCXYZBXYZD..').
>
> Instead this provides a dynamic programming greedy algorithm that finds
> the largest moved hunk and then has several modes on highlighting bounds.
>
> A note on the options '--submodule=diff' and '--color-words/--word-diff':
> In the conversion to use emit_line in the prior patches both submodules
> as well as word diff output carefully chose to call emit_line with sign=0.
> All output with sign=0 is ignored for move detection purposes in this
> patch, such that no weird looking output will be generated for these
> cases. This leads to another thought: We could pass on '--color-moved' to
> submodules such that they color up moved lines for themselves. If we'd do
> so only line moves within a repository boundary are marked up.
>
> Helped-by: Jonathan Tan <jonathantanmy@google.com>
> Signed-off-by: Stefan Beller <sbeller@google.com>
> Signed-off-by: Junio C Hamano <gitster@pobox.com>
> ---
> Documentation/config.txt       |  10 +-
> Documentation/diff-options.txt |  32 ++++
> color.h                        |   2 +
> diff.c                         | 342 +++++++++++++++++++++++++++++++++++--
> diff.h                         |  15 +-
> t/t4015-diff-whitespace.sh     | 373 
> +++++++++++++++++++++++++++++++++++++++++
> 6 files changed, 760 insertions(+), 14 deletions(-)
>
> diff --git a/Documentation/config.txt b/Documentation/config.txt
> index 475e874d51..73511a4603 100644
> --- a/Documentation/config.txt
> +++ b/Documentation/config.txt
> @@ -1051,14 +1051,20 @@ This does not affect linkgit:git-format-patch[1] 
> or the
> 'git-diff-{asterisk}' plumbing commands.  Can be overridden on the
> command line with the `--color[=<when>]` option.
>
> +diff.colorMoved::
> + If set moved lines in a diff are colored differently,
> + for details see '--color-moved' in linkgit:git-diff[1].
> +
> color.diff.<slot>::
>  Use customized color for diff colorization.  `<slot>` specifies
>  which part of the patch to use the specified color, and is one
>  of `context` (context text - `plain` is a historical synonym),
>  `meta` (metainformation), `frag`
>  (hunk header), 'func' (function in hunk header), `old` (removed lines),
> - `new` (added lines), `commit` (commit headers), or `whitespace`
> - (highlighting whitespace errors).
> + `new` (added lines), `commit` (commit headers), `whitespace`
> + (highlighting whitespace errors), `oldMoved`, `newMoved`,
> + `oldMovedAlternative` and `newMovedAlternative` (See the '<mode>'
> + setting of '--color-moved' in linkgit:git-diff[1] for details).
>
> color.decorate.<slot>::
>  Use customized color for 'git log --decorate' output.  `<slot>` is one
> diff --git a/Documentation/diff-options.txt 
> b/Documentation/diff-options.txt
> index 89cc0f48de..25259dbbc3 100644
> --- a/Documentation/diff-options.txt
> +++ b/Documentation/diff-options.txt
> @@ -231,6 +231,38 @@ ifdef::git-diff[]
> endif::git-diff[]
>  It is the same as `--color=never`.
>
> +--color-moved[=<mode>]::
> + Moved lines of code are colored differently.
> +ifdef::git-diff[]
> + It can be changed by the `diff.colorMoved` configuration setting.
> +endif::git-diff[]
> + The <mode> defaults to 'no' if the option is not given
> + and to 'adjacentbounds' if the option with no mode is given.
> + The mode must be one of:
> ++
> +--
> +no::
> + Moved lines are not highlighted.
> +nobounds::
> + Any line that is added in on location and was removed
s/on location/one location/

> + in another location will be colored with 'color.diff.newmoved'.
> + Any line that is removed in on location and was added
s/on location/one location/

> + in another location will be colored with 'color.diff.oldmoved'.
> +allbounds::
> + Based on 'nobounds'. Additionally blocks of moved code are
> + detected and the first and last line of a block will be highlighted
> + using 'color.diff.newMovedAlternate' or
> + 'color.diff.oldMovedAlternate'.
> +adjacentbounds::
> + The same as 'allbounds' except that highlighting is only performed
> + at adjacent block boundaries of blocks that have the same sign.
> +alternate::
> + Based on 'nobounds'. Additionally blocks of moved code are
> + detected. If moved blocks are adjacent mark one of them with the
> + alternative move color using 'color.diff.newMovedAlternate' or
> + 'color.diff.oldMovedAlternate'.
> +--
> +
> --word-diff[=<mode>]::
>  Show a word diff, using the <mode> to delimit changed words.
>  By default, words are delimited by whitespace; see

--
Philip

> diff --git a/color.h b/color.h
> index 90627650fc..04b3b87929 100644
> --- a/color.h
> +++ b/color.h
> @@ -42,6 +42,8 @@ struct strbuf;
> #define GIT_COLOR_BG_BLUE "\033[44m"
> #define GIT_COLOR_BG_MAGENTA "\033[45m"
> #define GIT_COLOR_BG_CYAN "\033[46m"
> +#define GIT_COLOR_DI_IT_CYAN "\033[2;3;36m"
> +#define GIT_COLOR_DI_IT_MAGENTA "\033[2;3;35m"
>
> /* A special value meaning "no color selected" */
> #define GIT_COLOR_NIL "NIL"
> diff --git a/diff.c b/diff.c
> index a3c16ef827..efd2530a89 100644
> --- a/diff.c
> +++ b/diff.c
> @@ -31,6 +31,7 @@ static int diff_indent_heuristic; /* experimental */
> static int diff_rename_limit_default = 400;
> static int diff_suppress_blank_empty;
> static int diff_use_color_default = -1;
> +static int diff_color_moved_default;
> static int diff_context_default = 3;
> static int diff_interhunk_context_default;
> static const char *diff_word_regex_cfg;
> @@ -55,6 +56,10 @@ static char diff_colors[][COLOR_MAXLEN] = {
>  GIT_COLOR_YELLOW, /* COMMIT */
>  GIT_COLOR_BG_RED, /* WHITESPACE */
>  GIT_COLOR_NORMAL, /* FUNCINFO */
> + GIT_COLOR_DI_IT_MAGENTA,/* OLD_MOVED */
> + GIT_COLOR_BG_RED, /* OLD_MOVED ALTERNATIVE */
> + GIT_COLOR_DI_IT_CYAN, /* NEW_MOVED */
> + GIT_COLOR_BG_GREEN, /* NEW_MOVED ALTERNATIVE */
> };
>
> static NORETURN void die_want_option(const char *option_name)
> @@ -80,6 +85,14 @@ static int parse_diff_color_slot(const char *var)
>  return DIFF_WHITESPACE;
>  if (!strcasecmp(var, "func"))
>  return DIFF_FUNCINFO;
> + if (!strcasecmp(var, "oldmoved"))
> + return DIFF_FILE_OLD_MOVED;
> + if (!strcasecmp(var, "oldmovedalternative"))
> + return DIFF_FILE_OLD_MOVED_ALT;
> + if (!strcasecmp(var, "newmoved"))
> + return DIFF_FILE_NEW_MOVED;
> + if (!strcasecmp(var, "newmovedalternative"))
> + return DIFF_FILE_NEW_MOVED_ALT;
>  return -1;
> }
>
> @@ -228,12 +241,35 @@ int git_diff_heuristic_config(const char *var, const 
> char *value, void *cb)
>  return 0;
> }
>
> +static int parse_color_moved(const char *arg)
> +{
> + if (!strcmp(arg, "no"))
> + return MOVED_LINES_NO;
> + else if (!strcmp(arg, "nobounds"))
> + return MOVED_LINES_BOUNDARY_NO;
> + else if (!strcmp(arg, "allbounds"))
> + return MOVED_LINES_BOUNDARY_ALL;
> + else if (!strcmp(arg, "adjacentbounds"))
> + return MOVED_LINES_BOUNDARY_ADJACENT;
> + else if (!strcmp(arg, "alternate"))
> + return MOVED_LINES_ALTERNATE;
> + else
> + return -1;
> +}
> +
> int git_diff_ui_config(const char *var, const char *value, void *cb)
> {
>  if (!strcmp(var, "diff.color") || !strcmp(var, "color.diff")) {
>  diff_use_color_default = git_config_colorbool(var, value);
>  return 0;
>  }
> + if (!strcmp(var, "diff.colormoved")) {
> + int cm = parse_color_moved(value);
> + if (cm < 0)
> + return -1;
> + diff_color_moved_default = cm;
> + return 0;
> + }
>  if (!strcmp(var, "diff.context")) {
>  diff_context_default = git_config_int(var, value);
>  if (diff_context_default < 0)
> @@ -354,6 +390,88 @@ int git_diff_basic_config(const char *var, const char 
> *value, void *cb)
>  return git_default_config(var, value, cb);
> }
>
> +struct moved_entry {
> + struct hashmap_entry ent;
> + const struct diff_line *line;
> + struct moved_entry *next_line;
> +};
> +
> +static void get_ws_cleaned_string(const struct diff_line *l,
> +   struct strbuf *out)
> +{
> + int i;
> + for (i = 0; i < l->len; i++) {
> + if (isspace(l->line[i]))
> + continue;
> + strbuf_addch(out, l->line[i]);
> + }
> +}
> +
> +static int diff_line_cmp_no_ws(const struct diff_line *a,
> + const struct diff_line *b,
> + const void *keydata)
> +{
> + int ret;
> + struct strbuf sba = STRBUF_INIT;
> + struct strbuf sbb = STRBUF_INIT;
> +
> + get_ws_cleaned_string(a, &sba);
> + get_ws_cleaned_string(b, &sbb);
> + ret = sba.len != sbb.len || strncmp(sba.buf, sbb.buf, sba.len);
> +
> + strbuf_release(&sba);
> + strbuf_release(&sbb);
> + return ret;
> +}
> +
> +static int diff_line_cmp(const struct diff_line *a,
> +    const struct diff_line *b,
> +    const void *keydata)
> +{
> + return a->len != b->len || strncmp(a->line, b->line, a->len);
> +}
> +
> +static int moved_entry_cmp(const struct moved_entry *a,
> +    const struct moved_entry *b,
> +    const void *keydata)
> +{
> + return diff_line_cmp(a->line, b->line, keydata);
> +}
> +
> +static int moved_entry_cmp_no_ws(const struct moved_entry *a,
> + const struct moved_entry *b,
> + const void *keydata)
> +{
> + return diff_line_cmp_no_ws(a->line, b->line, keydata);
> +}
> +
> +static unsigned get_line_hash(struct diff_line *line, unsigned ignore_ws)
> +{
> + static struct strbuf sb = STRBUF_INIT;
> +
> + if (ignore_ws) {
> + strbuf_reset(&sb);
> + get_ws_cleaned_string(line, &sb);
> + return memhash(sb.buf, sb.len);
> + } else {
> + return memhash(line->line, line->len);
> + }
> +}
> +
> +static struct moved_entry *prepare_entry(struct diff_options *o,
> + int line_no)
> +{
> + struct moved_entry *ret = xmalloc(sizeof(*ret));
> + unsigned ignore_ws = DIFF_XDL_TST(o, IGNORE_WHITESPACE);
> + struct diff_line *l = &o->line_buffer[line_no];
> +
> + ret->ent.hash = get_line_hash(l, ignore_ws);
> + ret->line = l;
> + ret->next_line = NULL;
> +
> + return ret;
> +}
> +
> static char *quote_two(const char *one, const char *two)
> {
>  int need_one = quote_c_style(one, NULL, NULL, 1);
> @@ -516,6 +634,179 @@ static void check_blank_at_eof(mmfile_t *mf1, 
> mmfile_t *mf2,
>  ecbdata->blank_at_eof_in_postimage = (at - l2) + 1;
> }
>
> +static void add_lines_to_move_detection(struct diff_options *o,
> + struct hashmap *add_lines,
> + struct hashmap *del_lines)
> +{
> + struct moved_entry *prev_line = NULL;
> +
> + int n;
> + for (n = 0; n < o->line_buffer_nr; n++) {
> + int sign = 0;
> + struct hashmap *hm;
> + struct moved_entry *key;
> +
> + switch (o->line_buffer[n].sign) {
> + case '+':
> + sign = '+';
> + hm = add_lines;
> + break;
> + case '-':
> + sign = '-';
> + hm = del_lines;
> + break;
> + case ' ':
> + default:
> + prev_line = NULL;
> + continue;
> + }
> +
> + key = prepare_entry(o, n);
> + if (prev_line &&
> +     prev_line->line->sign == sign)
> + prev_line->next_line = key;
> +
> + hashmap_add(hm, key);
> + prev_line = key;
> + }
> +}
> +
> +static void mark_color_as_moved_single_line(struct diff_options *o,
> +     struct diff_line *l, int alt_color)
> +{
> + switch (l->sign) {
> + case '+':
> + l->set = diff_get_color_opt(o,
> + DIFF_FILE_NEW_MOVED + alt_color);
> + break;
> + case '-':
> + l->set = diff_get_color_opt(o,
> + DIFF_FILE_OLD_MOVED + alt_color);
> + break;
> + default:
> + die("BUG: we should have continued earlier?");
> + }
> +}
> +
> +static void mark_color_as_moved(struct diff_options *o,
> + struct hashmap *add_lines,
> + struct hashmap *del_lines)
> +{
> + struct moved_entry **pmb = NULL; /* potentially moved blocks */
> + struct diff_line *prev_line = NULL;
> + int pmb_nr = 0, pmb_alloc = 0;
> + int n, flipped_block = 0;
> +
> + for (n = 0; n < o->line_buffer_nr; n++) {
> + struct hashmap *hm = NULL;
> + struct moved_entry *key;
> + struct moved_entry *match = NULL;
> + struct diff_line *l = &o->line_buffer[n];
> + int i, lp, rp, adjacent_blocks = 0;
> +
> + /* Check for any match to color it as a move. */
> + switch (l->sign) {
> + case '+':
> + hm = del_lines;
> + key = prepare_entry(o, n);
> + match = hashmap_get(hm, key, o);
> + free(key);
> + break;
> + case '-':
> + hm = add_lines;
> + key = prepare_entry(o, n);
> + match = hashmap_get(hm, key, o);
> + free(key);
> + break;
> + default: ;
> + }
> +
> + if (!match) {
> + pmb_nr = 0;
> + if (prev_line &&
> +     o->color_moved == MOVED_LINES_BOUNDARY_ALL)
> + mark_color_as_moved_single_line(o, prev_line, 1);
> + prev_line = NULL;
> + continue;
> + }
> +
> + if (o->color_moved == MOVED_LINES_BOUNDARY_NO) {
> + mark_color_as_moved_single_line(o, l, 0);
> + continue;
> + }
> +
> + /* Check any potential block runs, advance each or nullify */
> + for (i = 0; i < pmb_nr; i++) {
> + struct moved_entry *p = pmb[i];
> + struct moved_entry *pnext = (p && p->next_line) ?
> + p->next_line : NULL;
> + if (pnext &&
> +     !diff_line_cmp(pnext->line, l, o)) {
> + pmb[i] = p->next_line;
> + } else {
> + pmb[i] = NULL;
> + }
> + }
> +
> + /* Shrink the set of potential block to the remaining running */
> + for (lp = 0, rp = pmb_nr - 1; lp <= rp;) {
> + while (lp < pmb_nr && pmb[lp])
> + lp++;
> + /* lp points at the first NULL now */
> +
> + while (rp > -1 && !pmb[rp])
> + rp--;
> + /* rp points at the last non-NULL */
> +
> + if (lp < pmb_nr && rp > -1 && lp < rp) {
> + pmb[lp] = pmb[rp];
> + pmb[rp] = NULL;
> + rp--;
> + lp++;
> + }
> + }
> +
> + /* Remember the number of running sets */
> + pmb_nr = rp + 1;
> +
> + if (pmb_nr == 0) {
> + /*
> + * This line is the start of a new block.
> + * Setup the set of potential blocks.
> + */
> + for (; match; match = hashmap_get_next(hm, match)) {
> + ALLOC_GROW(pmb, pmb_nr + 1, pmb_alloc);
> + pmb[pmb_nr++] = match;
> + }
> +
> + if (o->color_moved == MOVED_LINES_BOUNDARY_ALL) {
> + adjacent_blocks = 1;
> + } else {
> + /* Check if two blocks are adjacent */
> + adjacent_blocks = prev_line &&
> +   prev_line->sign == l->sign;
> + }
> + }
> +
> + if (o->color_moved == MOVED_LINES_ALTERNATE) {
> + if (adjacent_blocks)
> + flipped_block = (flipped_block + 1) % 2;
> + mark_color_as_moved_single_line(o, l, flipped_block);
> + } else {
> + /* MOVED_LINES_BOUNDARY_{ADJACENT, ALL} */
> + mark_color_as_moved_single_line(o, l, adjacent_blocks);
> + if (adjacent_blocks && prev_line)
> + prev_line->set = l->set;
> + }
> +
> + prev_line = l;
> + }
> + if (prev_line && o->color_moved == MOVED_LINES_BOUNDARY_ALL)
> + mark_color_as_moved_single_line(o, prev_line, 1);
> +
> + free(pmb);
> +}
> +
> static void emit_diff_line(struct diff_options *o,
>     struct diff_line *e)
> {
> @@ -3518,6 +3809,8 @@ void diff_setup(struct diff_options *options)
>  options->line_buffer = NULL;
>  options->line_buffer_nr = 0;
>  options->line_buffer_alloc = 0;
> +
> + options->color_moved = diff_color_moved_default;
> }
>
> void diff_setup_done(struct diff_options *options)
> @@ -3627,6 +3920,9 @@ void diff_setup_done(struct diff_options *options)
>
>  if (DIFF_OPT_TST(options, FOLLOW_RENAMES) && options->pathspec.nr != 1)
>  die(_("--follow requires exactly one pathspec"));
> +
> + if (!options->use_color || external_diff())
> + options->color_moved = 0;
> }
>
> static int opt_arg(const char *arg, int arg_short, const char *arg_long, 
> int *val)
> @@ -4051,7 +4347,19 @@ int diff_opt_parse(struct diff_options *options,
>  }
>  else if (!strcmp(arg, "--no-color"))
>  options->use_color = 0;
> - else if (!strcmp(arg, "--color-words")) {
> + else if (!strcmp(arg, "--color-moved"))
> + if (diff_color_moved_default)
> + options->color_moved = diff_color_moved_default;
> + else
> + options->color_moved = MOVED_LINES_BOUNDARY_ADJACENT;
> + else if (!strcmp(arg, "--no-color-moved"))
> + options->color_moved = MOVED_LINES_NO;
> + else if (skip_prefix(arg, "--color-moved=", &arg)) {
> + int cm = parse_color_moved(arg);
> + if (cm < 0)
> + die("bad --color-moved argument: %s", arg);
> + options->color_moved = cm;
> + } else if (!strcmp(arg, "--color-words")) {
>  options->use_color = 1;
>  options->word_diff = DIFF_WORDS_COLOR;
>  }
> @@ -4856,16 +5164,9 @@ static void diff_flush_patch_all_file_pairs(struct 
> diff_options *o)
> {
>  int i;
>  struct diff_queue_struct *q = &diff_queued_diff;
> - /*
> - * For testing purposes we want to make sure the diff machinery
> - * works completely with the buffer. If there is anything emitted
> - * outside the emit_diff_line, then the order is screwed
> - * up and the tests will fail.
> - *
> - * TODO (later in this series):
> - * We'll unset this flag in a later patch.
> - */
> - o->use_buffer = 1;
> +
> + if (o->color_moved)
> + o->use_buffer = 1;
>
>  for (i = 0; i < q->nr; i++) {
>  struct diff_filepair *p = q->queue[i];
> @@ -4874,6 +5175,24 @@ static void diff_flush_patch_all_file_pairs(struct 
> diff_options *o)
>  }
>
>  if (o->use_buffer) {
> + if (o->color_moved) {
> + struct hashmap add_lines, del_lines;
> + unsigned ignore_ws = DIFF_XDL_TST(o, IGNORE_WHITESPACE);
> +
> + hashmap_init(&del_lines, ignore_ws ?
> + (hashmap_cmp_fn)moved_entry_cmp_no_ws :
> + (hashmap_cmp_fn)moved_entry_cmp, 0);
> + hashmap_init(&add_lines, ignore_ws ?
> + (hashmap_cmp_fn)moved_entry_cmp_no_ws :
> + (hashmap_cmp_fn)moved_entry_cmp, 0);
> +
> + add_lines_to_move_detection(o, &add_lines, &del_lines);
> + mark_color_as_moved(o, &add_lines, &del_lines);
> +
> + hashmap_free(&add_lines, 0);
> + hashmap_free(&del_lines, 0);
> + }
> +
>  for (i = 0; i < o->line_buffer_nr; i++)
>  emit_diff_line(o, &o->line_buffer[i]);
>
> @@ -4962,6 +5281,7 @@ void diff_flush(struct diff_options *options)
>  if (!options->file)
>  die_errno("Could not open /dev/null");
>  options->close_file = 1;
> + options->color_moved = 0;
>  for (i = 0; i < q->nr; i++) {
>  struct diff_filepair *p = q->queue[i];
>  if (check_pair_status(p))
> diff --git a/diff.h b/diff.h
> index be51e8f867..d9fbafd383 100644
> --- a/diff.h
> +++ b/diff.h
> @@ -7,6 +7,7 @@
> #include "tree-walk.h"
> #include "pathspec.h"
> #include "object.h"
> +#include "hashmap.h"
>
> struct rev_info;
> struct diff_options;
> @@ -228,6 +229,14 @@ struct diff_options {
>
>  struct diff_line *line_buffer;
>  int line_buffer_nr, line_buffer_alloc;
> +
> + enum {
> + MOVED_LINES_NO = 0,
> + MOVED_LINES_BOUNDARY_NO = 1,
> + MOVED_LINES_BOUNDARY_ALL = 2,
> + MOVED_LINES_BOUNDARY_ADJACENT = 3,
> + MOVED_LINES_ALTERNATE = 4,
> + } color_moved;
> };
>
> /* Emit [line_prefix] [set] line [reset] */
> @@ -243,7 +252,11 @@ enum color_diff {
>  DIFF_FILE_NEW = 5,
>  DIFF_COMMIT = 6,
>  DIFF_WHITESPACE = 7,
> - DIFF_FUNCINFO = 8
> + DIFF_FUNCINFO = 8,
> + DIFF_FILE_OLD_MOVED = 9,
> + DIFF_FILE_OLD_MOVED_ALT = 10,
> + DIFF_FILE_NEW_MOVED = 11,
> + DIFF_FILE_NEW_MOVED_ALT = 12
> };
> const char *diff_get_color(int diff_use_color, enum color_diff ix);
> #define diff_get_color_opt(o, ix) \
> diff --git a/t/t4015-diff-whitespace.sh b/t/t4015-diff-whitespace.sh
> index 289806d0c7..d4bd082af7 100755
> --- a/t/t4015-diff-whitespace.sh
> +++ b/t/t4015-diff-whitespace.sh
> @@ -972,4 +972,377 @@ test_expect_success 'option overrides 
> diff.wsErrorHighlight' '
>
> '
>
> +test_expect_success 'detect moved code, complete file' '
> + git reset --hard &&
> + cat <<-\EOF >test.c &&
> + #include<stdio.h>
> + main()
> + {
> + printf("Hello World");
> + }
> + EOF
> + git add test.c &&
> + git commit -m "add main function" &&
> + git mv test.c main.c &&
> + test_config color.diff.oldMoved "normal red" &&
> + test_config color.diff.newMoved "normal green" &&
> + git diff HEAD --color-moved --no-renames | test_decode_color >actual &&
> + cat >expected <<-\EOF &&
> + <BOLD>diff --git a/main.c b/main.c<RESET>
> + <BOLD>new file mode 100644<RESET>
> + <BOLD>index 0000000..a986c57<RESET>
> + <BOLD>--- /dev/null<RESET>
> + <BOLD>+++ b/main.c<RESET>
> + <CYAN>@@ -0,0 +1,5 @@<RESET>
> + <BGREEN>+<RESET><BGREEN>#include<stdio.h><RESET>
> + <BGREEN>+<RESET><BGREEN>main()<RESET>
> + <BGREEN>+<RESET><BGREEN>{<RESET>
> + <BGREEN>+<RESET><BGREEN>printf("Hello World");<RESET>
> + <BGREEN>+<RESET><BGREEN>}<RESET>
> + <BOLD>diff --git a/test.c b/test.c<RESET>
> + <BOLD>deleted file mode 100644<RESET>
> + <BOLD>index a986c57..0000000<RESET>
> + <BOLD>--- a/test.c<RESET>
> + <BOLD>+++ /dev/null<RESET>
> + <CYAN>@@ -1,5 +0,0 @@<RESET>
> + <BRED>-#include<stdio.h><RESET>
> + <BRED>-main()<RESET>
> + <BRED>-{<RESET>
> + <BRED>-printf("Hello World");<RESET>
> + <BRED>-}<RESET>
> + EOF
> +
> + test_cmp expected actual
> +'
> +
> +test_expect_success 'detect moved code, inside file' '
> + git reset --hard &&
> + cat <<-\EOF >main.c &&
> + #include<stdio.h>
> + int stuff()
> + {
> + printf("Hello ");
> + printf("World\n");
> + }
> +
> + int secure_foo(struct user *u)
> + {
> + if (!u->is_allowed_foo)
> + return;
> + foo(u);
> + }
> +
> + int main()
> + {
> + foo();
> + }
> + EOF
> + cat <<-\EOF >test.c &&
> + #include<stdio.h>
> + int bar()
> + {
> + printf("Hello World, but different\n");
> + }
> +
> + int another_function()
> + {
> + bar();
> + }
> + EOF
> + git add main.c test.c &&
> + git commit -m "add main and test file" &&
> + cat <<-\EOF >main.c &&
> + #include<stdio.h>
> + int stuff()
> + {
> + printf("Hello ");
> + printf("World\n");
> + }
> +
> + int main()
> + {
> + foo();
> + }
> + EOF
> + cat <<-\EOF >test.c &&
> + #include<stdio.h>
> + int bar()
> + {
> + printf("Hello World, but different\n");
> + }
> +
> + int secure_foo(struct user *u)
> + {
> + if (!u->is_allowed_foo)
> + return;
> + foo(u);
> + }
> +
> + int another_function()
> + {
> + bar();
> + }
> + EOF
> + test_config color.diff.oldMoved "normal red" &&
> + test_config color.diff.newMoved "normal green" &&
> + test_config color.diff.oldMovedAlternative "bold red" &&
> + test_config color.diff.newMovedAlternative "bold green" &&
> + git diff HEAD --no-renames --color-moved| test_decode_color >actual &&
> + cat <<-\EOF >expected &&
> + <BOLD>diff --git a/main.c b/main.c<RESET>
> + <BOLD>index 27a619c..7cf9336 100644<RESET>
> + <BOLD>--- a/main.c<RESET>
> + <BOLD>+++ b/main.c<RESET>
> + <CYAN>@@ -5,13 +5,6 @@<RESET> <RESET>printf("Hello ");<RESET>
> + printf("World\n");<RESET>
> + }<RESET>
> + <RESET>
> + <BRED>-int secure_foo(struct user *u)<RESET>
> + <BRED>-{<RESET>
> + <BRED>-if (!u->is_allowed_foo)<RESET>
> + <BRED>-return;<RESET>
> + <BRED>-foo(u);<RESET>
> + <BRED>-}<RESET>
> + <BRED>-<RESET>
> + int main()<RESET>
> + {<RESET>
> + foo();<RESET>
> + <BOLD>diff --git a/test.c b/test.c<RESET>
> + <BOLD>index 1dc1d85..e34eb69 100644<RESET>
> + <BOLD>--- a/test.c<RESET>
> + <BOLD>+++ b/test.c<RESET>
> + <CYAN>@@ -4,6 +4,13 @@<RESET> <RESET>int bar()<RESET>
> + printf("Hello World, but different\n");<RESET>
> + }<RESET>
> + <RESET>
> + <BGREEN>+<RESET><BGREEN>int secure_foo(struct user *u)<RESET>
> + <BGREEN>+<RESET><BGREEN>{<RESET>
> + <BGREEN>+<RESET><BGREEN>if (!u->is_allowed_foo)<RESET>
> + <BGREEN>+<RESET><BGREEN>return;<RESET>
> + <BGREEN>+<RESET><BGREEN>foo(u);<RESET>
> + <BGREEN>+<RESET><BGREEN>}<RESET>
> + <BGREEN>+<RESET>
> + int another_function()<RESET>
> + {<RESET>
> + bar();<RESET>
> + EOF
> +
> + test_cmp expected actual
> +'
> +
> +test_expect_success 'detect permutations inside moved code' '
> + git reset --hard &&
> + cat <<-\EOF >lines.txt &&
> + line 1
> + line 2
> + line 3
> + line 4
> + line 5
> + line 6
> + line 7
> + line 8
> + line 9
> + line 10
> + line 11
> + line 12
> + line 13
> + line 14
> + line 15
> + line 16
> + EOF
> + git add lines.txt &&
> + git commit -m "add poetry" &&
> + cat <<-\EOF >lines.txt &&
> + line 4
> + line 5
> + line 6
> + line 7
> + line 8
> + line 9
> + line 1
> + line 2
> + line 3
> + line 14
> + line 15
> + line 16
> + line 10
> + line 11
> + line 12
> + line 13
> + EOF
> + test_config color.diff.oldMoved "magenta" &&
> + test_config color.diff.newMoved "cyan" &&
> + test_config color.diff.oldMovedAlternative "blue" &&
> + test_config color.diff.newMovedAlternative "yellow" &&
> +
> +
> + git diff HEAD --no-renames --color-moved=nobounds| test_decode_color 
>  >actual &&
> + cat <<-\EOF >expected &&
> + <BOLD>diff --git a/lines.txt b/lines.txt<RESET>
> + <BOLD>index 47ea9c3..ba96a38 100644<RESET>
> + <BOLD>--- a/lines.txt<RESET>
> + <BOLD>+++ b/lines.txt<RESET>
> + <CYAN>@@ -1,16 +1,16 @@<RESET>
> + <MAGENTA>-line 1<RESET>
> + <MAGENTA>-line 2<RESET>
> + <MAGENTA>-line 3<RESET>
> + line 4<RESET>
> + line 5<RESET>
> + line 6<RESET>
> + line 7<RESET>
> + line 8<RESET>
> + line 9<RESET>
> + <CYAN>+<RESET><CYAN>line 1<RESET>
> + <CYAN>+<RESET><CYAN>line 2<RESET>
> + <CYAN>+<RESET><CYAN>line 3<RESET>
> + <CYAN>+<RESET><CYAN>line 14<RESET>
> + <CYAN>+<RESET><CYAN>line 15<RESET>
> + <CYAN>+<RESET><CYAN>line 16<RESET>
> + line 10<RESET>
> + line 11<RESET>
> + line 12<RESET>
> + line 13<RESET>
> + <MAGENTA>-line 14<RESET>
> + <MAGENTA>-line 15<RESET>
> + <MAGENTA>-line 16<RESET>
> + EOF
> + test_cmp expected actual &&
> +
> + git diff HEAD --no-renames --color-moved=adjacentbounds| 
> test_decode_color >actual &&
> + cat <<-\EOF >expected &&
> + <BOLD>diff --git a/lines.txt b/lines.txt<RESET>
> + <BOLD>index 47ea9c3..ba96a38 100644<RESET>
> + <BOLD>--- a/lines.txt<RESET>
> + <BOLD>+++ b/lines.txt<RESET>
> + <CYAN>@@ -1,16 +1,16 @@<RESET>
> + <MAGENTA>-line 1<RESET>
> + <MAGENTA>-line 2<RESET>
> + <MAGENTA>-line 3<RESET>
> + line 4<RESET>
> + line 5<RESET>
> + line 6<RESET>
> + line 7<RESET>
> + line 8<RESET>
> + line 9<RESET>
> + <CYAN>+<RESET><CYAN>line 1<RESET>
> + <CYAN>+<RESET><CYAN>line 2<RESET>
> + <YELLOW>+<RESET><YELLOW>line 3<RESET>
> + <YELLOW>+<RESET><YELLOW>line 14<RESET>
> + <CYAN>+<RESET><CYAN>line 15<RESET>
> + <CYAN>+<RESET><CYAN>line 16<RESET>
> + line 10<RESET>
> + line 11<RESET>
> + line 12<RESET>
> + line 13<RESET>
> + <MAGENTA>-line 14<RESET>
> + <MAGENTA>-line 15<RESET>
> + <MAGENTA>-line 16<RESET>
> + EOF
> + test_cmp expected actual &&
> +
> + test_config diff.colorMoved alternate &&
> + git diff HEAD --no-renames --color-moved| test_decode_color >actual &&
> + cat <<-\EOF >expected &&
> + <BOLD>diff --git a/lines.txt b/lines.txt<RESET>
> + <BOLD>index 47ea9c3..ba96a38 100644<RESET>
> + <BOLD>--- a/lines.txt<RESET>
> + <BOLD>+++ b/lines.txt<RESET>
> + <CYAN>@@ -1,16 +1,16 @@<RESET>
> + <MAGENTA>-line 1<RESET>
> + <MAGENTA>-line 2<RESET>
> + <MAGENTA>-line 3<RESET>
> + line 4<RESET>
> + line 5<RESET>
> + line 6<RESET>
> + line 7<RESET>
> + line 8<RESET>
> + line 9<RESET>
> + <CYAN>+<RESET><CYAN>line 1<RESET>
> + <CYAN>+<RESET><CYAN>line 2<RESET>
> + <CYAN>+<RESET><CYAN>line 3<RESET>
> + <YELLOW>+<RESET><YELLOW>line 14<RESET>
> + <YELLOW>+<RESET><YELLOW>line 15<RESET>
> + <YELLOW>+<RESET><YELLOW>line 16<RESET>
> + line 10<RESET>
> + line 11<RESET>
> + line 12<RESET>
> + line 13<RESET>
> + <BLUE>-line 14<RESET>
> + <BLUE>-line 15<RESET>
> + <BLUE>-line 16<RESET>
> + EOF
> + test_cmp expected actual &&
> +
> + test_config diff.colorMoved allbounds &&
> + git diff HEAD --no-renames --color-moved| test_decode_color >actual &&
> + cat <<-\EOF >expected &&
> + <BOLD>diff --git a/lines.txt b/lines.txt<RESET>
> + <BOLD>index 47ea9c3..ba96a38 100644<RESET>
> + <BOLD>--- a/lines.txt<RESET>
> + <BOLD>+++ b/lines.txt<RESET>
> + <CYAN>@@ -1,16 +1,16 @@<RESET>
> + <BLUE>-line 1<RESET>
> + <MAGENTA>-line 2<RESET>
> + <BLUE>-line 3<RESET>
> + line 4<RESET>
> + line 5<RESET>
> + line 6<RESET>
> + line 7<RESET>
> + line 8<RESET>
> + line 9<RESET>
> + <YELLOW>+<RESET><YELLOW>line 1<RESET>
> + <CYAN>+<RESET><CYAN>line 2<RESET>
> + <YELLOW>+<RESET><YELLOW>line 3<RESET>
> + <YELLOW>+<RESET><YELLOW>line 14<RESET>
> + <CYAN>+<RESET><CYAN>line 15<RESET>
> + <YELLOW>+<RESET><YELLOW>line 16<RESET>
> + line 10<RESET>
> + line 11<RESET>
> + line 12<RESET>
> + line 13<RESET>
> + <BLUE>-line 14<RESET>
> + <MAGENTA>-line 15<RESET>
> + <BLUE>-line 16<RESET>
> + EOF
> + test_cmp expected actual
> +'
> +
> +test_expect_success 'move detection does not mess up colored words' '
> + cat <<-\EOF >text.txt &&
> + Lorem Ipsum is simply dummy text of the printing and typesetting 
> industry.
> + EOF
> + git add text.txt &&
> + git commit -a -m "clean state" &&
> + cat <<-\EOF >text.txt &&
> + simply Lorem Ipsum dummy is text of the typesetting and printing 
> industry.
> + EOF
> + git diff --color-moved --word-diff >actual &&
> + git diff --word-diff >expect &&
> + test_cmp expect actual
> +'
> +
> +test_expect_success 'move detection with submodules' '
> + test_create_repo bananas &&
> + echo ripe >bananas/recipe &&
> + git -C bananas add recipe &&
> + test_commit fruit &&
> + test_commit -C bananas recipe &&
> + git submodule add ./bananas &&
> + git add bananas &&
> + git commit -a -m "bananas are like a heavy library?" &&
> + echo foul >bananas/recipe &&
> + echo ripe >fruit.t &&
> +
> + git diff --submodule=diff --color-moved >actual &&
> +
> + # no move detection as the moved line is across repository boundaries.
> + test_decode_color <actual >decoded_actual &&
> + ! grep BGREEN decoded_actual &&
> + ! grep BRED decoded_actual &&
> +
> + # nor did we mess with it another way
> + git diff --submodule=diff | test_decode_color >expect &&
> + test_cmp expect decoded_actual
> +'
> +
> test_done
> -- 
> 2.13.0.17.gab62347cd9
> 


  reply index

Thread overview: 125+ messages in thread (expand / mbox.gz / Atom feed / [top])
     [not found] <20170523024048.16879-1-sbeller@google.com/>
2017-05-24 21:40 ` [PATCHv5 00/17] Diff machine: highlight moved lines Stefan Beller
2017-05-24 21:40   ` [PATCHv5 01/17] diff: readability fix Stefan Beller
2017-05-24 21:40   ` [PATCHv5 02/17] diff: move line ending check into emit_hunk_header Stefan Beller
2017-05-24 21:40   ` [PATCHv5 03/17] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-05-24 21:40   ` [PATCHv5 04/17] diff: introduce more flexible emit function Stefan Beller
2017-06-13 21:54     ` Jonathan Tan
2017-06-13 23:41       ` Stefan Beller
2017-06-13 23:46         ` Jonathan Tan
2017-05-24 21:40   ` [PATCHv5 05/17] diff.c: convert fn_out_consume to use emit_line Stefan Beller
2017-05-24 21:40   ` [PATCHv5 06/17] diff.c: convert builtin_diff to use emit_line_* Stefan Beller
2017-05-24 21:40   ` [PATCHv5 07/17] diff.c: convert emit_rewrite_diff " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 08/17] diff.c: convert emit_rewrite_lines " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 09/17] submodule.c: convert show_submodule_summary to use emit_line_fmt Stefan Beller
2017-05-24 21:40   ` [PATCHv5 10/17] diff.c: convert emit_binary_diff_body to use emit_line_* Stefan Beller
2017-05-24 21:40   ` [PATCHv5 11/17] diff.c: convert show_stats " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 12/17] diff.c: convert word diffing " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 13/17] diff.c: convert diff_flush " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 14/17] diff.c: convert diff_summary " Stefan Beller
2017-05-24 21:40   ` [PATCHv5 15/17] diff.c: emit_line includes whitespace highlighting Stefan Beller
2017-05-24 21:40   ` [PATCHv5 16/17] diff: buffer all output if asked to Stefan Beller
2017-05-25  2:26     ` Junio C Hamano
2017-05-25  5:34       ` Stefan Beller
2017-05-26  1:09         ` Junio C Hamano
2017-06-13 22:07     ` Jonathan Tan
2017-06-14  2:52       ` Stefan Beller
2017-05-24 21:40   ` [PATCHv5 17/17] diff.c: color moved lines differently Stefan Beller
2017-05-25  2:27     ` Junio C Hamano
2017-05-25  5:39       ` Stefan Beller
2017-05-25  6:44   ` [PATCHv5 00/17] Diff machine: highlight moved lines Junio C Hamano
2017-05-25 16:31     ` Stefan Beller
2017-05-26  1:20       ` Junio C Hamano
2017-05-26 19:30         ` Stefan Beller
2017-05-27  0:18           ` [PATCH 0/1] " Stefan Beller
2017-05-27  0:18             ` [PATCH 1/1] diff.c: color moved lines differently Stefan Beller
2017-05-27  7:05               ` Philip Oakley [this message]
2017-05-30 21:33                 ` Stefan Beller
2017-06-01  0:24                   ` [PATCH] " Stefan Beller
2017-06-13 22:51                     ` Jonathan Tan
2017-06-14 18:55                       ` Stefan Beller
2017-06-20  2:47                         ` [PATCH 00/26] reroll of sb/diff-color-moved Stefan Beller
2017-06-20  2:47                           ` [PATCH 01/26] diff.c: readability fix Stefan Beller
2017-06-20  2:47                           ` [PATCH 02/26] diff.c: move line ending check into emit_hunk_header Stefan Beller
2017-06-20  2:47                           ` [PATCH 03/26] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-06-20  2:47                           ` [PATCH 04/26] diff.c: introduce emit_diff_symbol Stefan Beller
2017-06-21 19:36                             ` Junio C Hamano
2017-06-21 19:46                               ` Stefan Beller
2017-06-21 20:26                                 ` Junio C Hamano
2017-06-21 21:13                                 ` Junio C Hamano
2017-06-21 21:23                                   ` Stefan Beller
2017-06-21 21:43                                     ` Junio C Hamano
2017-06-21 21:51                                       ` Stefan Beller
2017-06-21 21:45                                 ` Brandon Williams
2017-06-21 21:52                                   ` Junio C Hamano
2017-06-21 21:55                                     ` Brandon Williams
2017-06-20  2:47                           ` [PATCH 05/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_MARKER Stefan Beller
2017-06-20  2:47                           ` [PATCH 06/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_FRAGINFO Stefan Beller
2017-06-20  2:47                           ` [PATCH 07/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_NO_LF_EOF Stefan Beller
2017-06-20  2:47                           ` [PATCH 08/26] diff.c: migrate emit_line_checked to use emit_diff_symbol Stefan Beller
2017-06-21 20:05                             ` Junio C Hamano
2017-06-22 23:30                               ` Stefan Beller
2017-06-22 23:37                                 ` Stefan Beller
2017-06-23  4:56                                   ` Junio C Hamano
2017-06-20  2:47                           ` [PATCH 09/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_WORDS{_PORCELAIN} Stefan Beller
2017-06-20  2:48                           ` [PATCH 10/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_INCOMPLETE Stefan Beller
2017-06-20  2:48                           ` [PATCH 11/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_FILEPAIR Stefan Beller
2017-06-20 20:01                             ` Jonathan Tan
2017-06-21 20:09                               ` Junio C Hamano
2017-06-22 23:59                                 ` Stefan Beller
2017-06-20  2:48                           ` [PATCH 12/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_HEADER Stefan Beller
2017-06-20  2:48                           ` [PATCH 13/26] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_BINARY_FILES Stefan Beller
2017-06-21 20:13                             ` Junio C Hamano
2017-06-21 20:47                               ` Stefan Beller
2017-06-20  2:48                           ` [PATCH 14/26] diff.c: emit_diff_symbol learns DIFF_SYMBOL_REWRITE_DIFF Stefan Beller
2017-06-20  2:48                           ` [PATCH 15/26] submodule.c: migrate diff output to use emit_diff_symbol Stefan Beller
2017-06-20 20:09                             ` Jonathan Tan
2017-06-20  2:48                           ` [PATCH 16/26] diff.c: convert emit_binary_diff_body " Stefan Beller
2017-06-21 20:16                             ` Junio C Hamano
2017-06-20  2:48                           ` [PATCH 17/26] diff.c: convert show_stats " Stefan Beller
2017-06-21 21:39                             ` Brandon Williams
2017-06-21 22:16                               ` Stefan Beller
2017-06-20  2:48                           ` [PATCH 18/26] diff.c: convert word diffing " Stefan Beller
2017-06-20  2:48                           ` [PATCH 19/26] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_STAT_SEP Stefan Beller
2017-06-20  2:48                           ` [PATCH 20/26] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_SUMMARY Stefan Beller
2017-06-20  2:48                           ` [PATCH 21/26] diff.c: buffer all output if asked to Stefan Beller
2017-06-20  2:48                           ` [PATCH 22/26] diff.c: color moved lines differently Stefan Beller
2017-06-20 20:13                             ` Jonathan Tan
2017-06-20 20:57                               ` Stefan Beller
2017-06-20  2:48                           ` [PATCH 23/26] diff.c: color moved lines differently, plain mode Stefan Beller
2017-06-20  2:48                           ` [PATCH 24/26] diff.c: add dimming to moved line detection Stefan Beller
2017-06-21 20:23                             ` Junio C Hamano
2017-06-20  2:48                           ` [PATCH 25/26] diff: document the new --color-moved setting Stefan Beller
2017-06-20  2:48                           ` [showing-off RFC/PATCH 26/26] diff.c: have a "machine parseable" move coloring Stefan Beller
2017-06-20  2:50                             ` Stefan Beller
2017-06-23 21:43                               ` Ævar Arnfjörð Bjarmason
2017-06-21 21:51                             ` Brandon Williams
2017-06-21 21:55                               ` Junio C Hamano
2017-06-21 22:40                                 ` Stefan Beller
2017-06-23  1:28                           ` [PATCHv2 00/25] reroll of sb/diff-color-moved Stefan Beller
2017-06-23  1:28                             ` [PATCHv2 01/25] diff.c: readability fix Stefan Beller
2017-06-23  1:28                             ` [PATCHv2 02/25] diff.c: move line ending check into emit_hunk_header Stefan Beller
2017-06-23  1:28                             ` [PATCHv2 03/25] diff.c: factor out diff_flush_patch_all_file_pairs Stefan Beller
2017-06-23  1:28                             ` [PATCHv2 04/25] diff.c: introduce emit_diff_symbol Stefan Beller
2017-06-23 20:07                               ` Junio C Hamano
2017-06-23 20:13                                 ` Stefan Beller
2017-06-23  1:28                             ` [PATCHv2 05/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_MARKER Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 06/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_FRAGINFO Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 07/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_NO_LF_EOF Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 08/25] diff.c: migrate emit_line_checked to use emit_diff_symbol Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 09/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_WORDS[_PORCELAIN] Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 10/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_CONTEXT_INCOMPLETE Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 11/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_FILEPAIR_{PLUS, MINUS} Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 12/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_HEADER Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 13/25] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_BINARY_FILES Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 14/25] diff.c: emit_diff_symbol learns DIFF_SYMBOL_REWRITE_DIFF Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 15/25] submodule.c: migrate diff output to use emit_diff_symbol Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 16/25] diff.c: convert emit_binary_diff_body " Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 17/25] diff.c: convert show_stats " Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 18/25] diff.c: convert word diffing " Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 19/25] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_STAT_SEP Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 20/25] diff.c: emit_diff_symbol learns about DIFF_SYMBOL_SUMMARY Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 21/25] diff.c: buffer all output if asked to Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 22/25] diff.c: color moved lines differently Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 23/25] diff.c: color moved lines differently, plain mode Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 24/25] diff.c: add dimming to moved line detection Stefan Beller
2017-06-23  1:29                             ` [PATCHv2 25/25] diff: document the new --color-moved setting Stefan Beller

Reply instructions:

You may reply publically 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 to all the recipients using the --to, --cc,
  and --in-reply-to switches of git-send-email(1):

  git send-email \
    --in-reply-to=39D67E58AC02490FB6D6124BDF3EE2A9@PhilipOakley \
    --to=philipoakley@iee.org \
    --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 \
    --cc=sbeller@google.com \
    /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

git@vger.kernel.org mailing list mirror (one of many)

Archives are clonable:
	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://ou63pmih66umazou.onion/inbox.comp.version-control.git
	nntp://czquwvybam4bgbro.onion/inbox.comp.version-control.git
	nntp://hjrcffqmbrq6wope.onion/inbox.comp.version-control.git
	nntp://news.gmane.org/gmane.comp.version-control.git

 note: .onion URLs require Tor: https://www.torproject.org/
       or Tor2web: https://www.tor2web.org/

AGPL code for this site: git clone https://public-inbox.org/ public-inbox