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

Re: [PATCH 1/2] run-command: factor out child_process_clear()

From
Stefan Beller <sbeller@google.com>
Date
Oct 26, 2015, 19:23 UTC
Message-ID
<CAGZ79kYopHL5D_7+bJV2P9GeKDHufPDBTo3a8bRL=4B3OmTGjg@mail.gmail.com>
In-Reply-To
<xmqqziz5h3n5.fsf@gitster.mtv.corp.google.com>
On Mon, Oct 26, 2015 at 11:43 AM, Junio C Hamano <gitster@pobox.com> wrote:
Show 10 quoted lines
> René Scharfe <l.s.r@web.de> writes:
>
>> Avoid duplication by moving the code to release allocated memory for
>> arguments and environment to its own function, child_process_clear().
>> Export it to provide a counterpart to child_process_init().
>>
>> Signed-off-by: Rene Scharfe <l.s.r@web.de>
>> ---
>
> Hmm, is this _deinit() Stefan added to his series recently?

Yes. Although you (Junio) take credit for actually using it in these places, too. :)

Previous: Junio C HamanoNext: René Scharfe
Message 5 of 7 in “run-command: factor out child_process_clear()”
  1. 1/2 run-command: factor out child_process_clear()René Scharfe, Oct 24, 2015
  2. 2/2 daemon: plug memory leakRené Scharfe, Oct 24, 2015
  3. Jeff KingOct 26, 2015
  4. Junio C HamanoOct 26, 2015
  5. Stefan BellerOct 26, 2015
  6. René ScharfeOct 27, 2015
  7. Stefan BellerOct 27, 2015

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.