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

Re: [PATCH] commit: Add commit_list prefix to reduce_heads function.

From
Thiago Farina <tfransosi@gmail.com>
Date
Dec 5, 2010, 17:29 UTC
Message-ID
<AANLkTikL4BWtzNgx1+MBYxRRdfL=Gu71KPjaiKXprvnb@mail.gmail.com>
In-Reply-To
<20101205170919.GA7913@burratino>
On Sun, Dec 5, 2010 at 3:09 PM, Jonathan Nieder <jrnieder@gmail.com> wrote:
Show 8 quoted lines
>  - "At my office there is a style guide indicating that each function
>   should live in a module with some other functions and be named to
>   indicate so (like perf, with its sched__* etc functions).  The idea
>   is that code with a simple high-level structure tends to be easier
>   to understand and we need to understand the code we use.  Can we
>   start changing the code to fit this style guide, so there is less
>   resistance to using it at my office?"
>

For me that is a good reason and I think it matches with what I had in mind but didn't write. Thanks for pointing it out.

Previous: Jonathan NiederNext: Thiago Farina
Message 5 of 13 in “commit: Add commit_list prefix to reduce_heads function.”
  1. commit: Add commit_list prefix to reduce_heads function.Thiago Farina, Dec 5, 2010
  2. Jonathan NiederDec 5, 2010
  3. Thiago FarinaDec 5, 2010
  4. Jonathan NiederDec 5, 2010
  5. Thiago FarinaDec 5, 2010
  6. Thiago FarinaDec 5, 2010
  7. Junio C HamanoDec 5, 2010
  8. Thiago FarinaDec 5, 2010
  9. Junio C HamanoDec 6, 2010
  10. Junio C HamanoDec 6, 2010
  11. Junio C HamanoDec 5, 2010
  12. Thiago FarinaDec 5, 2010
  13. Junio C HamanoDec 5, 2010

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.