git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 07/10] diff.c: convert fn_out_consume to use emit_line_*

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 13, 2016, 00:25 UTC
Message-ID
<xmqqk2egwsq3.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<1473572530-25764-8-git-send-email-stefanbeller@gmail.com>
Stefan Beller <stefanbeller@gmail.com> writes:
Show 45 quoted lines
> diff --git a/diff.c b/diff.c
> index 2aefd0f..7dcef73 100644
> --- a/diff.c
> +++ b/diff.c
> @@ -493,6 +493,19 @@ static void emit_line(struct diff_options *o, const char *set, const char *reset
>  	emit_line_0(o, set, reset, line[0], line+1, len-1);
>  }
>  
> +static void emit_line_fmt(struct diff_options *o,
> +			  const char *set, const char *reset,
> +			  const char *fmt, ...)
> +{
> +	struct strbuf sb = STRBUF_INIT;
> +	va_list ap;
> +	va_start(ap, fmt);
> +	strbuf_vaddf(&sb, fmt, ap);
> +	va_end(ap);
> +
> +	emit_line(o, set, reset, sb.buf, sb.len);
> +}
> +
>  static int new_blank_line_at_eof(struct emit_callback *ecbdata, const char *line, int len)
>  {
>  	if (!((ecbdata->ws_rule & WS_BLANK_AT_EOF) &&
> @@ -1217,7 +1230,6 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
>  	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);
>  
>  	o->found_changes = 1;
>  
> @@ -1233,10 +1245,12 @@ static void fn_out_consume(void *priv, char *line, unsigned long len)
>  		name_a_tab = strchr(ecbdata->label_path[0], ' ') ? "\t" : "";
>  		name_b_tab = strchr(ecbdata->label_path[1], ' ') ? "\t" : "";
>  
> -		fprintf(o->file, "%s%s--- %s%s%s\n",
> -			line_prefix, meta, ecbdata->label_path[0], reset, name_a_tab);
> -		fprintf(o->file, "%s%s+++ %s%s%s\n",
> -			line_prefix, meta, ecbdata->label_path[1], reset, name_b_tab);
> +		emit_line_fmt(o, meta, reset, "--- %s%s\n",
> +			      ecbdata->label_path[0], name_a_tab);
> +
> +		emit_line_fmt(o, meta, reset, "+++ %s%s\n",
> +			      ecbdata->label_path[1], name_b_tab);
Hmph, the original showed the following for the name-a line:
	diff_line_prefix(o) META "--- " label_path RESET name_a_tab LF

The updated one calls emit_line_fmt() with o, meta, reset, fmt and args, and then

 * strbuf_vaddf(&buf, "--- %s%s\n", label_path, name_a_tab) creates
   a string "--- " + label_path + LF
 * emit_line() is called on the whole thing with META and RESET
 * which is emit_line_0() that encloses the whole thing between META
   and RESET but knows the trailing LF should come after RESET.

So the coloring seems to be correct, but I am not sure where the line-prefix went.

Previous: Stefan BellerNext: Stefan Beller
Message 14 of 21 in “Another preparatory series for rename detection”
  1. 00/10 Another preparatory series for rename detectionStefan Beller, Sep 11, 2016
  2. 01/10 diff: move line ending check into emit_hunk_headerStefan Beller, Sep 11, 2016
  3. Junio C HamanoSep 12, 2016
  4. 02/10 diff: emit_{add, del, context}_line to increase {pre,post}image line countStefan Beller, Sep 11, 2016
  5. Junio C HamanoSep 12, 2016
  6. 03/10 diff.c: drop tautologous condition in emit_line_0Stefan Beller, Sep 11, 2016
  7. Junio C HamanoSep 12, 2016
  8. Stefan BellerSep 16, 2016
  9. 05/10 diff.c: reintroduce diff_flush_patch for all filesStefan Beller, Sep 11, 2016
  10. Junio C HamanoSep 13, 2016
  11. 04/10 diff.c: rename diff_flush_patch to diff_flush_patch_filepairStefan Beller, Sep 11, 2016
  12. Junio C HamanoSep 12, 2016
  13. 07/10 diff.c: convert fn_out_consume to use emit_line_*Stefan Beller, Sep 11, 2016
  14. Junio C HamanoSep 13, 2016
  15. Stefan BellerSep 13, 2016
  16. 10/10 submodule.c: convert show_submodule_summary to use emit_line_fmtStefan Beller, Sep 11, 2016
  17. 09/10 diff.c: convert emit_rewrite_lines to use emit_line_0Stefan Beller, Sep 11, 2016
  18. 08/10 diff.c: convert emit_rewrite_diff to use emit_line_*Stefan Beller, Sep 11, 2016
  19. 06/10 diff.c: emit_line_0 can handle no colorStefan Beller, Sep 11, 2016
  20. Junio C HamanoSep 13, 2016
  21. Stefan BellerSep 13, 2016

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.