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

Re: [PATCH] t5000: use check_mtime()

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 27, 2023, 22:45 UTC
Message-ID
<xmqq5yaln795.fsf@gitster.g>
In-Reply-To
<a47f8172-a8d0-abe8-09a8-f31b9e19b1a1@web.de>
René Scharfe <l.s.r@web.de> writes:
Show 8 quoted lines
> Care to send a patch?
>
> There's more by the way:
> ...
> Personally I'd like to avoid wasting space and the second form looks odd
> to me, which turned me into a repeat offender out of negligence, I
> guess.  Anyway, since we have a documented rule (that came about by
> majority vote of code lines IIUC) we should follow it.

I used to appreciate that I can grep for "func (" to find shell function definition while I can grep for "func(" for C, but that was a long time ago. "git grep <pattern> --" followed by "\*.c" vs "\*.sh" is what I use to limit the hits these days anyway ;-)

Previous: René Scharfe
Message 4 of 4 in “t5000: use check_mtime()”
  1. t5000: use check_mtime()René Scharfe, Mar 25, 2023
  2. Andrei RybakMar 26, 2023
  3. René ScharfeMar 27, 2023
  4. Junio C HamanoMar 27, 2023

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.