Skip to content

Commit 0d5466d

Browse files
pcloudsgitster
authored andcommitted
git.c: make it clear save_env() is for alias handling only
Signed-off-by: Nguyễn Thái Ngọc Duy <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent ec3de38 commit 0d5466d

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

git.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,14 +25,14 @@ static const char *env_names[] = {
2525
GIT_PREFIX_ENVIRONMENT
2626
};
2727
static char *orig_env[4];
28-
static int saved_environment;
28+
static int saved_env_before_alias;
2929

30-
static void save_env(void)
30+
static void save_env_before_alias(void)
3131
{
3232
int i;
33-
if (saved_environment)
33+
if (saved_env_before_alias)
3434
return;
35-
saved_environment = 1;
35+
saved_env_before_alias = 1;
3636
orig_cwd = xgetcwd();
3737
for (i = 0; i < ARRAY_SIZE(env_names); i++) {
3838
orig_env[i] = getenv(env_names[i]);
@@ -233,6 +233,7 @@ static int handle_alias(int *argcp, const char ***argv)
233233
char *alias_string;
234234
int unused_nongit;
235235

236+
save_env_before_alias();
236237
subdir = setup_git_directory_gently(&unused_nongit);
237238

238239
alias_command = (*argv)[0];
@@ -527,7 +528,7 @@ static void handle_builtin(int argc, const char **argv)
527528

528529
builtin = get_builtin(cmd);
529530
if (builtin) {
530-
if (saved_environment && (builtin->option & NO_SETUP))
531+
if (saved_env_before_alias && (builtin->option & NO_SETUP))
531532
restore_env();
532533
else
533534
exit(run_builtin(builtin, argc, argv));
@@ -587,7 +588,6 @@ static int run_argv(int *argcp, const char ***argv)
587588
*/
588589
if (done_alias)
589590
break;
590-
save_env();
591591
if (!handle_alias(argcp, argv))
592592
break;
593593
done_alias = 1;

0 commit comments

Comments
 (0)