@matheusbsbs / matheus-bagliano / commits / a3c158d4a5
Fork 0

Add a prefix output callback to diff output

The callback can be used to add some prefix string to each line of diff output. Signed-off-by: Bo Yang <struggleyb.nku@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>

Bo Yang committed May 26, 2010 at 15:08 UTC a3c158d4a58b17c1e4a8d3f793344beee21d3a4c
2 files changed +40 -27
diff.c
+35 -27
@@ -194,8 +194,8 @@ struct emit_callback {
194 sane_truncate_fn truncate;
195 const char **label_path;
196 struct diff_words_data *diff_words;
197 + struct diff_options *opt;
198 int *found_changesp;
198 - FILE *file;
199 struct strbuf *header;
200 };
201
@@ -282,11 +282,19 @@ static void check_blank_at_eof(mmfile_t *mf1, mmfile_t *mf2,
282 ecbdata->blank_at_eof_in_postimage = (at - l2) + 1;
283 }
284
285 -static void emit_line_0(FILE *file, const char *set, const char *reset,
285 +static void emit_line_0(struct diff_options *o, const char *set, const char *reset,
286 int first, const char *line, int len)
287 {
288 int has_trailing_newline, has_trailing_carriage_return;
289 int nofirst;
290 + FILE *file = o->file;
291 +
292 + if (o->output_prefix) {
293 + struct strbuf *msg = NULL;
294 + msg = o->output_prefix(o, o->output_prefix_data);
295 + assert(msg);
296 + fwrite(msg->buf, msg->len, 1, file);
297 + }
298
299 if (len == 0) {
300 has_trailing_newline = (first == '\n');
@@ -316,10 +324,10 @@ static void emit_line_0(FILE *file, const char *set, const char *reset,
324 fputc('\n', file);
325 }
326
319 -static void emit_line(FILE *file, const char *set, const char *reset,
327 +static void emit_line(struct diff_options *o, const char *set, const char *reset,
328 const char *line, int len)
329 {
322 - emit_line_0(file, set, reset, line[0], line+1, len-1);
330 + emit_line_0(o, set, reset, line[0], line+1, len-1);
331 }
332
333 static int new_blank_line_at_eof(struct emit_callback *ecbdata, const char *line, int len)
@@ -341,15 +349,15 @@ static void emit_add_line(const char *reset,
349 const char *set = diff_get_color(ecbdata->color_diff, DIFF_FILE_NEW);
350
351 if (!*ws)
344 - emit_line_0(ecbdata->file, set, reset, '+', line, len);
352 + emit_line_0(ecbdata->opt, set, reset, '+', line, len);
353 else if (new_blank_line_at_eof(ecbdata, line, len))
354 /* Blank line at EOF - paint '+' as well */
347 - emit_line_0(ecbdata->file, ws, reset, '+', line, len);
355 + emit_line_0(ecbdata->opt, ws, reset, '+', line, len);
356 else {
357 /* Emit just the prefix, then the rest. */
350 - emit_line_0(ecbdata->file, set, reset, '+', "", 0);
358 + emit_line_0(ecbdata->opt, set, reset, '+', "", 0);
359 ws_check_emit(line, len, ecbdata->ws_rule,
352 - ecbdata->file, set, reset, ws);
360 + ecbdata->opt->file, set, reset, ws);
361 }
362 }
363
@@ -370,23 +378,23 @@ static void emit_hunk_header(struct emit_callback *ecbdata,
378 if (len < 10 ||
379 memcmp(line, atat, 2) ||
380 !(ep = memmem(line + 2, len - 2, atat, 2))) {
373 - emit_line(ecbdata->file, plain, reset, line, len);
381 + emit_line(ecbdata->opt, plain, reset, line, len);
382 return;
383 }
384 ep += 2; /* skip over @@ */
385
386 /* The hunk header in fraginfo color */
379 - emit_line(ecbdata->file, frag, reset, line, ep - line);
387 + emit_line(ecbdata->opt, frag, reset, line, ep - line);
388
389 /* blank before the func header */
390 for (cp = ep; ep - line < len; ep++)
391 if (*ep != ' ' && *ep != '\t')
392 break;
393 if (ep != cp)
386 - emit_line(ecbdata->file, plain, reset, cp, ep - cp);
394 + emit_line(ecbdata->opt, plain, reset, cp, ep - cp);
395
396 if (ep < line + len)
389 - emit_line(ecbdata->file, func, reset, ep, line + len - ep);
397 + emit_line(ecbdata->opt, func, reset, ep, line + len - ep);
398 }
399
400 static struct diff_tempfile *claim_diff_tempfile(void) {
@@ -446,7 +454,7 @@ static void emit_rewrite_lines(struct emit_callback *ecb,
454 len = endp ? (endp - data + 1) : size;
455 if (prefix != '+') {
456 ecb->lno_in_preimage++;
449 - emit_line_0(ecb->file, old, reset, '-',
457 + emit_line_0(ecb->opt, old, reset, '-',
458 data, len);
459 } else {
460 ecb->lno_in_postimage++;
@@ -458,7 +466,7 @@ static void emit_rewrite_lines(struct emit_callback *ecb,
466 if (!endp) {
467 const char *plain = diff_get_color(ecb->color_diff,
468 DIFF_PLAIN);
461 - emit_line_0(ecb->file, plain, reset, '\\',
469 + emit_line_0(ecb->opt, plain, reset, '\\',
470 nneof, strlen(nneof));
471 }
472 }
@@ -508,7 +516,7 @@ static void emit_rewrite_diff(const char *name_a,
516 ecbdata.color_diff = color_diff;
517 ecbdata.found_changesp = &o->found_changes;
518 ecbdata.ws_rule = whitespace_rule(name_b ? name_b : name_a);
511 - ecbdata.file = o->file;
519 + ecbdata.opt = o;
520 if (ecbdata.ws_rule & WS_BLANK_AT_EOF) {
521 mmfile_t mf1, mf2;
522 mf1.ptr = (char *)data_one;
@@ -840,7 +848,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
848 const char *reset = diff_get_color(ecbdata->color_diff, DIFF_RESET);
849
850 if (ecbdata->header) {
843 - fprintf(ecbdata->file, "%s", ecbdata->header->buf);
851 + fprintf(ecbdata->opt->file, "%s", ecbdata->header->buf);
852 strbuf_reset(ecbdata->header);
853 ecbdata->header = NULL;
854 }
@@ -852,9 +860,9 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
860 name_a_tab = strchr(ecbdata->label_path[0], ' ') ? "\t" : "";
861 name_b_tab = strchr(ecbdata->label_path[1], ' ') ? "\t" : "";
862
855 - fprintf(ecbdata->file, "%s--- %s%s%s\n",
863 + fprintf(ecbdata->opt->file, "%s--- %s%s%s\n",
864 meta, ecbdata->label_path[0], reset, name_a_tab);
857 - fprintf(ecbdata->file, "%s+++ %s%s%s\n",
865 + fprintf(ecbdata->opt->file, "%s+++ %s%s%s\n",
866 meta, ecbdata->label_path[1], reset, name_b_tab);
867 ecbdata->label_path[0] = ecbdata->label_path[1] = NULL;
868 }
@@ -872,15 +880,15 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
880 find_lno(line, ecbdata);
881 emit_hunk_header(ecbdata, line, len);
882 if (line[len-1] != '\n')
875 - putc('\n', ecbdata->file);
883 + putc('\n', ecbdata->opt->file);
884 return;
885 }
886
887 if (len < 1) {
880 - emit_line(ecbdata->file, reset, reset, line, len);
888 + emit_line(ecbdata->opt, reset, reset, line, len);
889 if (ecbdata->diff_words
890 && ecbdata->diff_words->type == DIFF_WORDS_PORCELAIN)
883 - fputs("~\n", ecbdata->file);
891 + fputs("~\n", ecbdata->opt->file);
892 return;
893 }
894
@@ -896,11 +904,11 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
904 }
905 diff_words_flush(ecbdata);
906 if (ecbdata->diff_words->type == DIFF_WORDS_PORCELAIN) {
899 - emit_line(ecbdata->file, plain, reset, line, len);
900 - fputs("~\n", ecbdata->file);
907 + emit_line(ecbdata->opt, plain, reset, line, len);
908 + fputs("~\n", ecbdata->opt->file);
909 } else {
910 /* don't print the prefix character */
903 - emit_line(ecbdata->file, plain, reset, line+1, len-1);
911 + emit_line(ecbdata->opt, plain, reset, line+1, len-1);
912 }
913 return;
914 }
@@ -912,7 +920,7 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
920 ecbdata->lno_in_preimage++;
921 if (line[0] == ' ')
922 ecbdata->lno_in_postimage++;
915 - emit_line(ecbdata->file, color, reset, line, len);
923 + emit_line(ecbdata->opt, color, reset, line, len);
924 } else {
925 ecbdata->lno_in_postimage++;
926 emit_add_line(reset, ecbdata, line + 1, len - 1);
@@ -1477,7 +1485,7 @@ static void checkdiff_consume(void *priv, char *line, unsigned long len)
1485 fprintf(data->o->file, "%s:%d: %s.\n",
1486 data->filename, data->lineno, err);
1487 free(err);
1480 - emit_line(data->o->file, set, reset, line, 1);
1488 + emit_line(data->o, set, reset, line, 1);
1489 ws_check_emit(line + 1, len - 1, data->ws_rule,
1490 data->o->file, set, reset, ws);
1491 } else if (line[0] == ' ') {
@@ -1787,7 +1795,7 @@ static void builtin_diff(const char *name_a,
1795 ecbdata.ws_rule = whitespace_rule(name_b ? name_b : name_a);
1796 if (ecbdata.ws_rule & WS_BLANK_AT_EOF)
1797 check_blank_at_eof(&mf1, &mf2, &ecbdata);
1790 - ecbdata.file = o->file;
1798 + ecbdata.opt = o;
1799 ecbdata.header = header.len ? &header : NULL;
1800 xpp.flags = XDF_NEED_MINIMAL | o->xdl_opts;
1801 xecfg.ctxlen = o->context;
diff.h
+5
@@ -9,6 +9,7 @@
9 struct rev_info;
10 struct diff_options;
11 struct diff_queue_struct;
12 +struct strbuf;
13
14 typedef void (*change_fn_t)(struct diff_options *options,
15 unsigned old_mode, unsigned new_mode,
@@ -25,6 +26,8 @@ typedef void (*add_remove_fn_t)(struct diff_options *options,
26 typedef void (*diff_format_fn_t)(struct diff_queue_struct *q,
27 struct diff_options *options, void *data);
28
29 +typedef struct strbuf *(*diff_prefix_fn_t)(struct diff_options *opt, void *data);
30 +
31 #define DIFF_FORMAT_RAW 0x0001
32 #define DIFF_FORMAT_DIFFSTAT 0x0002
33 #define DIFF_FORMAT_NUMSTAT 0x0004
@@ -130,6 +133,8 @@ struct diff_options {
133 add_remove_fn_t add_remove;
134 diff_format_fn_t format_callback;
135 void *format_callback_data;
136 + diff_prefix_fn_t output_prefix;
137 + void *output_prefix_data;
138 };
139
140 enum color_diff {