Re: [PATCH v2 12/23] query_refspecs(): move some constants out of the loop

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Michael Haggerty wrote:
> diff --git a/remote.c b/remote.c
> index e42daa8..a5e6c7f 100644
> --- a/remote.c
> +++ b/remote.c
> @@ -825,6 +825,8 @@ static int query_refspecs(struct refspec *refs, int ref_count, struct refspec *q
>  {
>         int i;
>         int find_src = !query->src;
> +       const char *needle = find_src ? query->dst : query->src;
> +       char **result = find_src ? &query->src : &query->dst;
>
>         if (find_src && !query->dst)
>                 return error("query_refspecs: need either src or dst");
> @@ -833,8 +835,6 @@ static int query_refspecs(struct refspec *refs, int ref_count, struct refspec *q
>                 struct refspec *refspec = &refs[i];
>                 const char *key = find_src ? refspec->dst : refspec->src;
>                 const char *value = find_src ? refspec->src : refspec->dst;
> -               const char *needle = find_src ? query->dst : query->src;
> -               char **result = find_src ? &query->src : &query->dst;
>
>                 if (!refspec->dst)
>                         continue;

Why? Is it used in some later patch?
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]