Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxx> writes:
> Signed-off-by: Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxx>
> ---
> connect.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/connect.c b/connect.c
> index da89c9c..d4051dd 100644
> --- a/connect.c
> +++ b/connect.c
> @@ -179,7 +179,6 @@ static int count_refspec_match(const char *pattern,
> for (weak_match = match = 0; refs; refs = refs->next) {
> char *name = refs->name;
> int namelen = strlen(name);
> - int weak_match;
>
> if (namelen < patlen ||
> memcmp(name + namelen - patlen, pattern, patlen))
This one is an obvious bug. Essentially, it makes weak matches
ignored. Unfortunately this has been hiding a larger bug in the
caller of this function. I am refactoring the mess right now.
-
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
