Merge branch 'jk/color-diff-plain-is-context' into maint
"color.diff.plain" was a misnomer; give it 'color.diff.context' as a more logical synonym. * jk/color-diff-plain-is-context: diff.h: rename DIFF_PLAIN color slot to DIFF_CONTEXT diff: accept color.diff.context as a synonym for "plain"
This commit is contained in:
28
diff.c
28
diff.c
@ -42,7 +42,7 @@ static long diff_algorithm;
|
||||
|
||||
static char diff_colors[][COLOR_MAXLEN] = {
|
||||
GIT_COLOR_RESET,
|
||||
GIT_COLOR_NORMAL, /* PLAIN */
|
||||
GIT_COLOR_NORMAL, /* CONTEXT */
|
||||
GIT_COLOR_BOLD, /* METAINFO */
|
||||
GIT_COLOR_CYAN, /* FRAGINFO */
|
||||
GIT_COLOR_RED, /* OLD */
|
||||
@ -54,8 +54,8 @@ static char diff_colors[][COLOR_MAXLEN] = {
|
||||
|
||||
static int parse_diff_color_slot(const char *var)
|
||||
{
|
||||
if (!strcasecmp(var, "plain"))
|
||||
return DIFF_PLAIN;
|
||||
if (!strcasecmp(var, "context") || !strcasecmp(var, "plain"))
|
||||
return DIFF_CONTEXT;
|
||||
if (!strcasecmp(var, "meta"))
|
||||
return DIFF_METAINFO;
|
||||
if (!strcasecmp(var, "frag"))
|
||||
@ -501,7 +501,7 @@ static void emit_add_line(const char *reset,
|
||||
static void emit_hunk_header(struct emit_callback *ecbdata,
|
||||
const char *line, int len)
|
||||
{
|
||||
const char *plain = diff_get_color(ecbdata->color_diff, DIFF_PLAIN);
|
||||
const char *context = diff_get_color(ecbdata->color_diff, DIFF_CONTEXT);
|
||||
const char *frag = diff_get_color(ecbdata->color_diff, DIFF_FRAGINFO);
|
||||
const char *func = diff_get_color(ecbdata->color_diff, DIFF_FUNCINFO);
|
||||
const char *reset = diff_get_color(ecbdata->color_diff, DIFF_RESET);
|
||||
@ -518,7 +518,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, plain, reset, line, len);
|
||||
emit_line(ecbdata->opt, context, reset, line, len);
|
||||
return;
|
||||
}
|
||||
ep += 2; /* skip over @@ */
|
||||
@ -540,7 +540,7 @@ static void emit_hunk_header(struct emit_callback *ecbdata,
|
||||
if (*ep != ' ' && *ep != '\t')
|
||||
break;
|
||||
if (ep != cp) {
|
||||
strbuf_addstr(&msgbuf, plain);
|
||||
strbuf_addstr(&msgbuf, context);
|
||||
strbuf_add(&msgbuf, cp, ep - cp);
|
||||
strbuf_addstr(&msgbuf, reset);
|
||||
}
|
||||
@ -623,10 +623,10 @@ static void emit_rewrite_lines(struct emit_callback *ecb,
|
||||
data += len;
|
||||
}
|
||||
if (!endp) {
|
||||
const char *plain = diff_get_color(ecb->color_diff,
|
||||
DIFF_PLAIN);
|
||||
const char *context = diff_get_color(ecb->color_diff,
|
||||
DIFF_CONTEXT);
|
||||
putc('\n', ecb->opt->file);
|
||||
emit_line_0(ecb->opt, plain, reset, '\\',
|
||||
emit_line_0(ecb->opt, context, reset, '\\',
|
||||
nneof, strlen(nneof));
|
||||
}
|
||||
}
|
||||
@ -1086,7 +1086,7 @@ static void init_diff_words_data(struct emit_callback *ecbdata,
|
||||
struct diff_words_style *st = ecbdata->diff_words->style;
|
||||
st->old.color = diff_get_color_opt(o, DIFF_FILE_OLD);
|
||||
st->new.color = diff_get_color_opt(o, DIFF_FILE_NEW);
|
||||
st->ctx.color = diff_get_color_opt(o, DIFF_PLAIN);
|
||||
st->ctx.color = diff_get_color_opt(o, DIFF_CONTEXT);
|
||||
}
|
||||
}
|
||||
|
||||
@ -1162,7 +1162,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
|
||||
{
|
||||
struct emit_callback *ecbdata = priv;
|
||||
const char *meta = diff_get_color(ecbdata->color_diff, DIFF_METAINFO);
|
||||
const char *plain = diff_get_color(ecbdata->color_diff, DIFF_PLAIN);
|
||||
const char *context = diff_get_color(ecbdata->color_diff, DIFF_CONTEXT);
|
||||
const char *reset = diff_get_color(ecbdata->color_diff, DIFF_RESET);
|
||||
struct diff_options *o = ecbdata->opt;
|
||||
const char *line_prefix = diff_line_prefix(o);
|
||||
@ -1233,7 +1233,7 @@ 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(ecbdata->opt, plain, reset, line, len);
|
||||
emit_line(ecbdata->opt, context, reset, line, len);
|
||||
fputs("~\n", ecbdata->opt->file);
|
||||
} else {
|
||||
/*
|
||||
@ -1245,7 +1245,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
|
||||
line++;
|
||||
len--;
|
||||
}
|
||||
emit_line(ecbdata->opt, plain, reset, line, len);
|
||||
emit_line(ecbdata->opt, context, reset, line, len);
|
||||
}
|
||||
return;
|
||||
}
|
||||
@ -1253,7 +1253,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
|
||||
if (line[0] != '+') {
|
||||
const char *color =
|
||||
diff_get_color(ecbdata->color_diff,
|
||||
line[0] == '-' ? DIFF_FILE_OLD : DIFF_PLAIN);
|
||||
line[0] == '-' ? DIFF_FILE_OLD : DIFF_CONTEXT);
|
||||
ecbdata->lno_in_preimage++;
|
||||
if (line[0] == ' ')
|
||||
ecbdata->lno_in_postimage++;
|
||||
|
Reference in New Issue
Block a user