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

Re: [PATCH 1/4] "log" and friends: --children option

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Apr 3, 2008, 19:58 UTC
Message-ID
<alpine.LSU.1.00.0804032158040.4008@racer.site>
In-Reply-To
<7vlk3v2tg9.fsf@gitster.siamese.dyndns.org>
Hi,
On Thu, 3 Apr 2008, Junio C Hamano wrote:
Show 13 quoted lines
> @@ -1398,6 +1404,26 @@ int setup_revisions(int argc, const char **argv, struct rev_info *revs, const ch
>  	return left;
>  }
>  
> +static void add_child(struct rev_info *revs, struct commit *parent, struct commit *child)
> +{
> +	struct commit_list *l = xcalloc(1, sizeof(*l));
> +
> +	l->item = child;
> +	l->next = add_decoration(&revs->children, &parent->object, l);
> +}
> +
> +static void set_children(struct rev_info *revs)
I'd call this add_children().

Other than that, nicely done! I did not think it was possible with as few changes.

Ciao, Dscho

Previous: Junio C HamanoNext: Junio C Hamano
Message 3 of 8 in “Blame in reverse”
  1. 0/4 Blame in reverseJunio C Hamano, Apr 3, 2008
  2. 1/4 "log" and friends: --children optionJunio C Hamano, Apr 3, 2008
  3. Johannes SchindelinApr 3, 2008
  4. 2/4 builtin-blame.c: move prepare_final() into a separate function.Junio C Hamano, Apr 3, 2008
  5. 3/4 builtin-blame.c: allow more than 16 parentsJunio C Hamano, Apr 3, 2008
  6. 4/4 git-blame --reverseJunio C Hamano, Apr 3, 2008
  7. Johannes SixtApr 3, 2008
  8. Junio C HamanoApr 3, 2008

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.