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

Re: [PATCH] git-mailsplit: with maildirs try to process new/ if cur/ is empty

From
Jakub Narebski <jnareb@gmail.com>
Date
Nov 5, 2007, 12:58 UTC
Message-ID
<fgn429$gs9$1@ger.gmane.org>
In-Reply-To
<20071105124920.17726.qmail@746e9cce42b49f.315fe32.mid.smarden.org>
[Cc: Gerrit Pape <pape@smarden.org>, git@vger.kernel.org]
Gerrit Pape wrote:
> +       char *sub[] = { "cur", "new" };
[...]
> +       for (i = 0; i < 2; ++i) {

Wouldn't it be better to use sizeof(sub)/sizeof(sub[0]) or it's macro equivalent ARRAY_SIZE(sub) instead of hardcoding 2 to avoid errors?

-- 
Jakub Narebski
Warsaw, Poland
ShadeHawk on #git
Previous: Gerrit PapeNext: Jeff King
Message 4 of 20 in “git-mailsplit: with maildirs try to process new/ if cur/ is empty”
  1. git-mailsplit: with maildirs try to process new/ if cur/ is emptyGerrit Pape, Oct 26, 2007
  2. Fernando J. PeredaOct 26, 2007
  3. git-mailsplit: with maildirs try to process new/ if cur/ is emptyGerrit Pape, Nov 5, 2007
  4. Jakub NarebskiNov 5, 2007
  5. Jeff KingNov 5, 2007
  6. Alex RiesenNov 5, 2007
  7. Michael CohenNov 6, 2007
  8. Alex RiesenNov 6, 2007
  9. Jeff KingNov 6, 2007
  10. Johannes SchindelinNov 6, 2007
  11. Jeff KingNov 6, 2007
  12. Johannes SchindelinNov 6, 2007
  13. Karl HasselströmNov 6, 2007
  14. Johannes SchindelinNov 6, 2007
  15. Alex RiesenNov 6, 2007
  16. git-mailsplit: with maildirs not only process cur/, but also new/Gerrit Pape, Nov 6, 2007
  17. Junio C HamanoNov 8, 2007
  18. Jeff KingNov 8, 2007
  19. Alex RiesenNov 8, 2007
  20. Fernando J. PeredaNov 8, 2007

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.