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

[PATCH] stash: use named constant when parsing "--all"

From
PWPhillip Wood <phillip.wood123@gmail.com>
Date
Oct 4, 2026, 10:05 UTC
Message-ID
<06b58ae0a1d81f4d1518eadecc3385072a72155a.1791108351.git.phillip.wood@dunelm.org.uk>
From: Phillip Wood <phillip.wood@dunelm.org.uk>

The code that stashes all untracked files compares the value of the "include_untracked" variable to the constant "INCLUDE_ALL_FILES", however the option parsing code for "--all" uses a hard coded integer instead. Replace the integer with the named constant.

Signed-off-by: Phillip Wood <phillip.wood@dunelm.org.uk>
---
base-commit: a018953688f1b10bddf91bff8747068f5f4746a4
Published-As: https://github.com/phillipwood/git/releases/tag/pw%2Fstash-all-untracked-use-named-constant%2Fv1
View-Changes-At: https://github.com/phillipwood/git/compare/a01895368...06b58ae0a
Fetch-It-Via: git fetch https://github.com/phillipwood/git pw/stash-all-untracked-use-named-constant/v1
 builtin/stash.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/builtin/stash.c b/builtin/stash.c
index 7a9843413b1..4bdd51cd49f 100644
--- a/builtin/stash.c
+++ b/builtin/stash.c
@@ -1932,7 +1932,7 @@ static int push_stash(int argc, const char **argv, const char *prefix,
 		OPT_BOOL('u', "include-untracked", &include_untracked,
 			 N_("include untracked files in stash")),
 		OPT_SET_INT('a', "all", &include_untracked,
-			    N_("include ignore files"), 2),
+			    N_("include ignore files"), INCLUDE_ALL_FILES),
 		OPT_STRING('m', "message", &stash_msg, N_("message"),
 			   N_("stash message")),
 		OPT_PATHSPEC_FROM_FILE(&pathspec_from_file),
@@ -2039,7 +2039,7 @@ static int save_stash(int argc, const char **argv, const char *prefix,
 		OPT_BOOL('u', "include-untracked", &include_untracked,
 			 N_("include untracked files in stash")),
 		OPT_SET_INT('a', "all", &include_untracked,
-			    N_("include ignore files"), 2),
+			    N_("include ignore files"), INCLUDE_ALL_FILES),
 		OPT_STRING('m', "message", &stash_msg, "message",
 			   N_("stash message")),
 		OPT_END()
-- 
2.56.0.134.g299a3c16181
Next: Junio C Hamano
Message 1 of 2 in “stash: use named constant when parsing "--all"”
  1. stash: use named constant when parsing "--all"Phillip Wood, Oct 4, 2026
  2. Junio C HamanoOct 5, 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.