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

Re: [PATCH 2/3] rev-list: remove last static vars used in "show_commit"

From
Paolo Bonzini <bonzini@gnu.org>
Date
Apr 8, 2009, 07:40 UTC
Message-ID
<49DC54DC.3060408@gnu.org>
In-Reply-To
<20090407040854.4338.40055.chriscool@tuxfamily.org>
Show 6 quoted lines
>  	struct rev_info revs;
> +	struct rev_list_info info;
>  	int reaches = 0, all = 0;
>  
> +	memset(&info, 0, sizeof(info));
> +	info.revs = &revs;

Would it make sense to embed the struct rev_info entirely in the new struct, without going through a pointer?

Paolo
Previous: Christian CouderNext: Christian Couder
Message 4 of 8 in “list-objects: add "void *data" parameter to show functions”
  1. 1/3 list-objects: add "void *data" parameter to show functionsChristian Couder, Apr 6, 2009
  2. 2/3 rev-list: remove last static vars used in "show_commit"Christian Couder, Apr 6, 2009
  3. 3/3 rev-list: add "int bisect_show_flags" in "struct rev_list_info"Christian Couder, Apr 7, 2009
  4. Paolo BonziniApr 8, 2009
  5. Christian CouderApr 9, 2009
  6. Junio C HamanoApr 8, 2009
  7. Christian CouderApr 8, 2009
  8. Junio C HamanoApr 8, 2009

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.