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

Re: What's cooking in git.git (Aug 2008, #07; Sat, 23)

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 25, 2008, 20:19 UTC
Message-ID
<7vtzd8g851.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<20080825163203.GD22184@cuci.nl>
"Stephen R. van den Berg" <srb@cuci.nl> writes:
Show 15 quoted lines
> Junio C Hamano wrote:
> ...
>>It seems that kill_some_child() will not kill anything if nobody else is
>>coming from the same address, while the old code did kill some.  Is this
>>intended?
>
> This is intended.
> ...
>>...  If you
>>simplify add_child() to queue the newborn always at the front of the list,
>>your kill_some_child() will continue to do so, so I do not see the point
>>of the loop in add_child().  Am I mistaken?
> ...
> You are mistaken.
> The point is that we need to find out *duplicate* IP-adresses.
Lightbulb goes on on both counts.  Thanks.
> Shall I incorporate your suggested changes (as far as I consider them ok)
> into my patches and resubmit them?

I think your response makes the only remaining is the style fixup, which was my [1/3] from yesterday, so I think we are Ok with what already we have.

Thanks.
Previous: Stephen R. van den BergNext: Stephen R. van den Berg
Message 10 of 11 in “What's cooking in git.git (Aug 2008, #07; Sat, 23)”
  1. Junio C HamanoAug 24, 2008
  2. Maintaining "needswork" section of "What's (not) cooking"Junio C Hamano, Aug 24, 2008
  3. Johannes SchindelinAug 24, 2008
  4. Junio C HamanoAug 24, 2008
  5. Junio C HamanoAug 24, 2008
  6. 1/3 daemon.c: minor style fixupJunio C Hamano, Aug 24, 2008
  7. 2/3 daemon.c: simplify add_child() and kill_some_child() logicJunio C Hamano, Aug 24, 2008
  8. 3/3 daemon.c: make sure kill_some_child() really kills somebodyJunio C Hamano, Aug 24, 2008
  9. Stephen R. van den BergAug 25, 2008
  10. Junio C HamanoAug 25, 2008
  11. Stephen R. van den BergAug 25, 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.