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

[PATCH v10 1/2] git-pull.c: introduce git_pull_config()

From
Mehul Jain <mehul.jain2029@gmail.com>
Date
Mar 21, 2016, 18:18 UTC
Message-ID
<1458584283-23816-2-git-send-email-mehul.jain2029@gmail.com>
In-Reply-To
<1458584283-23816-1-git-send-email-mehul.jain2029@gmail.com>

git-pull makes a seperate call to git_config_get_bool() to read the value of "rebase.autostash". This can be reduced as a call to git_config() is already there in the code.

Introduce a callback function git_pull_config() to read "rebase.autostash" along with other variables.

Helped-by: Junio C Hamano <gitster@pobox.com>
Helped-by: Paul Tan <pyokagan@gmail.com>
Helped-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Mehul Jain <mehul.jain2029@gmail.com>
---
 builtin/pull.c | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)
diff --git a/builtin/pull.c b/builtin/pull.c
index 10eff03..c21897d 100644
--- a/builtin/pull.c
+++ b/builtin/pull.c
@@ -86,6 +86,7 @@ static char *opt_commit;
 static char *opt_edit;
 static char *opt_ff;
 static char *opt_verify_signatures;
+static int config_autostash;
 static struct argv_array opt_strategies = ARGV_ARRAY_INIT;
 static struct argv_array opt_strategy_opts = ARGV_ARRAY_INIT;
 static char *opt_gpg_sign;
@@ -306,6 +307,18 @@ static enum rebase_type config_get_rebase(void)
 }
 
 /**
+ * Read config variables.
+ */
+static int git_pull_config(const char *var, const char *value, void *cb)
+{
+	if (!strcmp(var, "rebase.autostash")) {
+		config_autostash = git_config_bool(var, value);
+		return 0;
+	}
+	return git_default_config(var, value, cb);
+}
+
+/**
  * Returns 1 if there are unstaged changes, 0 otherwise.
  */
 static int has_unstaged_changes(const char *prefix)
@@ -823,7 +836,7 @@ int cmd_pull(int argc, const char **argv, const char *prefix)
 	if (opt_rebase < 0)
 		opt_rebase = config_get_rebase();
 
-	git_config(git_default_config, NULL);
+	git_config(git_pull_config, NULL);
 
 	if (read_cache_unmerged())
 		die_resolve_conflict("Pull");
@@ -835,12 +848,11 @@ int cmd_pull(int argc, const char **argv, const char *prefix)
 		hashclr(orig_head);
 
 	if (opt_rebase) {
-		int autostash = 0;
+		int autostash = config_autostash;
 
 		if (is_null_sha1(orig_head) && !is_cache_unborn())
 			die(_("Updating an unborn branch with changes added to the index."));
 
-		git_config_get_bool("rebase.autostash", &autostash);
 		if (!autostash)
 			die_on_unclean_work_tree(prefix);
 
-- 
2.7.1.340.g69eb491.dirty
Previous: Mehul JainNext: Mehul Jain
Message 2 of 16 in “introduce --[no-]autostash command line flag”
  1. 0/2 introduce --[no-]autostash command line flagMehul Jain, Mar 21, 2016
  2. 1/2 git-pull.c: introduce git_pull_config()Mehul Jain, Mar 21, 2016
  3. 2/2 pull --rebase: add --[no-]autostash flagMehul Jain, Mar 21, 2016
  4. Matthieu MoyMar 21, 2016
  5. 2/2 pull --rebase: add --[no-]autostash flagMehul Jain, Mar 21, 2016
  6. Eric SunshineMar 25, 2016
  7. Eric SunshineMar 25, 2016
  8. Eric SunshineMar 25, 2016
  9. Mehul JainMar 25, 2016
  10. Eric SunshineMar 25, 2016
  11. Matthieu MoyMar 25, 2016
  12. Mehul JainMar 25, 2016
  13. Matthieu MoyMar 25, 2016
  14. Mehul JainMar 25, 2016
  15. Eric SunshineMar 25, 2016
  16. Mehul JainMar 25, 2016

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.