[PATCH] git: make was_alias non-static

Alexander Kuleshov· Mar 1, 2015, 06:48 UTC · lore

'was_alias' variable does not need to store it's value each iteration in the loop, anyway this variable changes it's value with run_argv. So it does not need to be static.

Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>
---
 git.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to git.c +1 −1

diff --git a/git.c b/git.c
index 1780233..b8b6520 100644
--- a/git.c
+++ b/git.c
@@ -682,7 +682,7 @@ int main(int argc, char **av)
 
 	while (1) {
 		static int done_help = 0;
-		static int was_alias = 0;
+		int was_alias = 0;
 		was_alias = run_argv(&argc, &argv);
 		if (errno != ENOENT)
 			break;

-- 
2.3.1.422.ge618558.dirty

Eric Sunshine· Mar 1, 2015, 23:44 UTC · re: Alexander Kuleshov · lore

Re: [PATCH] git: make was_alias non-static

On Sun, Mar 1, 2015 at 1:48 AM, Alexander Kuleshov <kuleshovmail@gmail.com> wrote:

Show 17 quoted lines

> 'was_alias' variable does not need to store it's value each iteration in the
> loop, anyway this variable changes it's value with run_argv. So it does not
> need to be static.
>
> Signed-off-by: Alexander Kuleshov <kuleshovmail@gmail.com>
> ---
>  git.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/git.c b/git.c
> index 1780233..b8b6520 100644
> --- a/git.c
> +++ b/git.c
> @@ -682,7 +682,7 @@ int main(int argc, char **av)
>
>         while (1) {
>                 static int done_help = 0;

'done_help' could receive similar treatment by dropping 'static' and moving it outside the 'while' loop. Doing so would make the logic easier to understand.

> -               static int was_alias = 0;
> +               int was_alias = 0;

No need for the 0 initialization since 'was_alias' is being assigned immediately below.

Show 5 quoted lines

>                 was_alias = run_argv(&argc, &argv);
>                 if (errno != ENOENT)
>                         break;
> --
> 2.3.1.422.ge618558.dirty

patchgit: make was_alias non-static