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

[PATCH 0/2] remove unnecessary if statement

From
Usman Akinyemi <usmanakinyemi202@gmail.com>
Date
Apr 20, 2025, 02:54 UTC
Message-ID
<20250420025501.183290-1-usmanakinyemi202@gmail.com>

Since we already teach the `repo_config()` in "f29f1990b5 (config: teach repo_config to allow `repo` to be NULL, 2025-03-08)" to allow `repo` to be NULL, no need to check if `repo` is NULL before calling `repo_config()`.

This was suggested by Patrick Steinhardt <ps@pks.im>
Usman Akinyemi (2):
  builtin/add: remove unnecessary if statement
  builtin/difftool: remove unnecessary if statement
 builtin/add.c      | 3 +--
 builtin/difftool.c | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)
-- 
2.49.0
Next: Usman Akinyemi
Message 1 of 3 in “remove unnecessary if statement”
  1. 0/2 remove unnecessary if statementUsman Akinyemi, Apr 20, 2025
  2. 1/2 builtin/add: remove unnecessary if statementUsman Akinyemi, Apr 20, 2025
  3. 2/2 builtin/difftool: remove unnecessary if statementUsman Akinyemi, Apr 20, 2025

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.