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

[PATCH v2 04/13] config: format int64s gently

From
Derrick Stolee via GitGitGadget <gitgitgadget@gmail.com>
Date
Feb 13, 2026, 23:55 UTC
Message-ID
<2bca4d231686e33ea9d4d85b10fcffd60a63ad46.1771026918.git.gitgitgadget@gmail.com>
In-Reply-To
<pull.2044.v2.git.1771026918.gitgitgadget@gmail.com>
From: Derrick Stolee <stolee@gmail.com>

Move the logic for formatting int64 config values into a helper method and use gentle parsing when needed.

Signed-off-by: Derrick Stolee <stolee@gmail.com>
---
 builtin/config.c | 27 +++++++++++++++++++++++----
 1 file changed, 23 insertions(+), 4 deletions(-)
diff --git a/builtin/config.c b/builtin/config.c
index 4c4c791883..d259a91d53 100644
--- a/builtin/config.c
+++ b/builtin/config.c
@@ -237,6 +237,25 @@ struct strbuf_list {
 	int alloc;
 };
 
+static int format_config_int64(struct strbuf *buf,
+			       const char *key_,
+			       const char *value_,
+			       const struct key_value_info *kvi,
+			       int gently)
+{
+	int64_t v = 0;
+	if (gently) {
+		if (git_parse_int64(value_, &v))
+			return -1;
+	} else {
+		/* may die() */
+		v = git_config_int64(key_, value_ ? value_ : "", kvi);
+	}
+
+	strbuf_addf(buf, "%"PRId64, v);
+	return 0;
+}
+
 /*
  * Format the configuration key-value pair (`key_`, `value_`) and
  * append it into strbuf `buf`.  Returns a negative value on failure,
@@ -249,8 +268,9 @@ struct strbuf_list {
 static int format_config(const struct config_display_options *opts,
 			 struct strbuf *buf, const char *key_,
 			 const char *value_, const struct key_value_info *kvi,
-			 int gently UNUSED)
+			 int gently)
 {
+	int res = 0;
 	if (opts->show_scope)
 		show_config_scope(opts, kvi, buf);
 	if (opts->show_origin)
@@ -262,8 +282,7 @@ static int format_config(const struct config_display_options *opts,
 			strbuf_addch(buf, opts->key_delim);
 
 		if (opts->type == TYPE_INT)
-			strbuf_addf(buf, "%"PRId64,
-				    git_config_int64(key_, value_ ? value_ : "", kvi));
+			res = format_config_int64(buf, key_, value_, kvi, gently);
 		else if (opts->type == TYPE_BOOL)
 			strbuf_addstr(buf, git_config_bool(key_, value_) ?
 				      "true" : "false");
@@ -309,7 +328,7 @@ static int format_config(const struct config_display_options *opts,
 		}
 	}
 	strbuf_addch(buf, opts->term);
-	return 0;
+	return res;
 }
 
 static int show_all_config(const char *key_, const char *value_,
-- 
gitgitgadget
Previous: Patrick SteinhardtNext: Junio C Hamano
Message 24 of 58 in “[RFC] Make 'git config list --type=' parse and filter types”
  1. 0/5 [RFC] Make 'git config list --type=' parse and filter typesDerrick Stolee via GitGitGadget, Feb 10, 2026
  2. 1/5 config: move show_all_config()Derrick Stolee via GitGitGadget, Feb 10, 2026
  3. 2/5 parse: add git_parse_maybe_pathname()Derrick Stolee via GitGitGadget, Feb 10, 2026
  4. Patrick SteinhardtFeb 11, 2026
  5. 3/5 config: allow format_config() to filterDerrick Stolee via GitGitGadget, Feb 10, 2026
  6. Junio C HamanoFeb 10, 2026
  7. Derrick StoleeFeb 10, 2026
  8. 4/5 config: create special init for list modeDerrick Stolee via GitGitGadget, Feb 10, 2026
  9. 5/5 config: make 'git config list --type=<X>' workDerrick Stolee via GitGitGadget, Feb 10, 2026
  10. Patrick SteinhardtFeb 11, 2026
  11. Derrick StoleeFeb 11, 2026
  12. Patrick SteinhardtFeb 12, 2026
  13. Junio C HamanoFeb 10, 2026
  14. Derrick StoleeFeb 10, 2026
  15. Patrick SteinhardtFeb 11, 2026
  16. 00/13 Make 'git config list --type=' parse and filter typesDerrick Stolee via GitGitGadget, Feb 13, 2026
  17. 01/13 config: move show_all_config()Derrick Stolee via GitGitGadget, Feb 13, 2026
  18. 02/13 config: add 'gently' parameter to format_config()Derrick Stolee via GitGitGadget, Feb 13, 2026
  19. Patrick SteinhardtFeb 17, 2026
  20. 03/13 config: make 'git config list --type=<X>' workDerrick Stolee via GitGitGadget, Feb 13, 2026
  21. Patrick SteinhardtFeb 17, 2026
  22. Junio C HamanoFeb 17, 2026
  23. Patrick SteinhardtFeb 17, 2026
  24. 04/13 config: format int64s gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  25. Junio C HamanoFeb 14, 2026
  26. Patrick SteinhardtFeb 17, 2026
  27. Derrick StoleeFeb 23, 2026
  28. 05/13 config: format bools gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  29. 06/13 config: format bools or ints gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  30. Patrick SteinhardtFeb 17, 2026
  31. Derrick StoleeFeb 23, 2026
  32. 07/13 config: format bools or strings in helperDerrick Stolee via GitGitGadget, Feb 13, 2026
  33. 08/13 parse: add git_parse_maybe_pathname()Derrick Stolee via GitGitGadget, Feb 13, 2026
  34. 09/13 config: format paths gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  35. Patrick SteinhardtFeb 17, 2026
  36. 10/13 config: format expiry dates gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  37. 11/13 color: add color_parse_gently()Derrick Stolee via GitGitGadget, Feb 13, 2026
  38. Patrick SteinhardtFeb 17, 2026
  39. Junio C HamanoFeb 17, 2026
  40. Derrick StoleeFeb 23, 2026
  41. Junio C HamanoFeb 23, 2026
  42. 12/13 config: format colors gentlyDerrick Stolee via GitGitGadget, Feb 13, 2026
  43. 13/13 config: restructure format_config()Derrick Stolee via GitGitGadget, Feb 13, 2026
  44. Patrick SteinhardtFeb 17, 2026
  45. 00/13 Make 'git config list --type=' parse and filter typesDerrick Stolee via GitGitGadget, Feb 23, 2026
  46. 01/13 config: move show_all_config()Derrick Stolee via GitGitGadget, Feb 23, 2026
  47. 02/13 config: add 'gently' parameter to format_config()Derrick Stolee via GitGitGadget, Feb 23, 2026
  48. 03/13 config: make 'git config list --type=<X>' workDerrick Stolee via GitGitGadget, Feb 23, 2026
  49. 04/13 config: format int64s gentlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  50. 05/13 config: format bools gentlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  51. 06/13 config: format bools or ints gentlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  52. 07/13 config: format bools or strings in helperDerrick Stolee via GitGitGadget, Feb 23, 2026
  53. 08/13 config: format paths gentlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  54. 09/13 config: format expiry dates quietlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  55. 10/13 color: add color_parse_quietly()Derrick Stolee via GitGitGadget, Feb 23, 2026
  56. 11/13 config: format colors quietlyDerrick Stolee via GitGitGadget, Feb 23, 2026
  57. 12/13 config: restructure format_config()Derrick Stolee via GitGitGadget, Feb 23, 2026
  58. 13/13 config: use an enum for typeDerrick Stolee via GitGitGadget, Feb 23, 2026

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.