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

Re: [PATCH 2/4] mergetool: move main program flow into a main() function

From
Johannes Sixt <j6t@kdbg.org>
Date
Oct 7, 2016, 05:51 UTC
Message-ID
<49cabceb-fdff-3ccb-b25d-ade9b08bdda2@kdbg.org>
In-Reply-To
<1475797679-32712-2-git-send-email-davvid@gmail.com>
Am 07.10.2016 um 01:47 schrieb David Aguilar:
> Signed-off-by: David Aguilar <davvid@gmail.com>
> ---

An answer to "why?" is missing here. I guess it is just so that everything happens in functions, and that there is only the invocation of main at the top-level of the script. I am unsure whether this is sufficient justification; you are adding a level of indentation after all.

-- Hannes
Previous: David AguilarNext: David Aguilar
Message 3 of 7 in “mergetool: add copyright”
  1. 1/4 mergetool: add copyrightDavid Aguilar, Oct 6, 2016
  2. 2/4 mergetool: move main program flow into a main() functionDavid Aguilar, Oct 6, 2016
  3. Johannes SixtOct 7, 2016
  4. 3/4 mergetool: honor diff.orderFileDavid Aguilar, Oct 6, 2016
  5. Johannes SixtOct 7, 2016
  6. 4/4 mergetool: honor -O<order-file>David Aguilar, Oct 6, 2016
  7. Johannes SixtOct 7, 2016

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.