Subject: Re: [PATCH 6/7] Register a callback for graph output.

From: Thomas Rast <[email-masked]>
Date: Mon, 17 May 2010 08:57:47 +0200

Bo Yang wrote:
> +static struct strbuf *diff_output_prefix_callback(FILE *file, int print, void *data)
> +{
> +	struct git_graph *graph = data;
> +	static struct strbuf msgbuf = STRBUF_INIT;
> +
> +	assert(graph);
> +
> +	strbuf_reset(&msgbuf);
> +	graph_padding_line(graph, &msgbuf);
> +	if (print) {
> +		fwrite(msgbuf.buf, sizeof(char), msgbuf.len, stdout);
> +	}
> +	return &msgbuf;
> +}
> +

Ok, this partially answers my last mail and means that a simple string
would be too tedious, so a callback is needed.

It leaks the 'msgbuf' in the 'printing' case though.

-- 
Thomas Rast
trast@{inf,student}.ethz.ch