# [PATCH] ref-filter: fix indentation

4 messages from 2015-07-29 to 2015-07-29. Participants: Matthieu Moy, Karthik Nayak, Junio C Hamano.
Thread: https://gitlist.dev/t/39964

## Matthieu Moy, 2015-07-29 15:52

Subject: [PATCH] ref-filter: fix indentation
Message-ID: <1438185150-31730-1-git-send-email-Matthieu.Moy@imag.fr>
URL: https://gitlist.dev/e/1438185150-31730-1-git-send-email-Matthieu.Moy%40imag.fr

```
Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
---
This is meant to be applied on top of kn/for-each-ref.

 ref-filter.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ref-filter.c b/ref-filter.c
index 43502a4..3fbbbeb 100644
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -868,8 +868,8 @@ static int ref_filter_handler(const char *refname, const struct object_id *oid,
 	struct ref_array_item *ref;
 
 	if (flag & REF_BAD_NAME) {
-		  warning("ignoring ref with broken name %s", refname);
-		  return 0;
+		warning("ignoring ref with broken name %s", refname);
+		return 0;
 	}
 
 	if (*filter->name_patterns && !match_name_as_path(filter->name_patterns, refname))
-- 
2.5.0.rc0.7.ge1edd74.dirty

```

## Karthik Nayak, 2015-07-29 16:11

Subject: Re: [PATCH] ref-filter: fix indentation
Message-ID: <CAOLa=ZQf+_BkvjFC4Fv=X8_3A=KAs7e4JB2CZC3n5U6yffSLpg@mail.gmail.com>
URL: https://gitlist.dev/e/CAOLa%3DZQf%2B_BkvjFC4Fv%3DX8_3A%3DKAs7e4JB2CZC3n5U6yffSLpg%40mail.gmail.com
In-Reply-To: <1438185150-31730-1-git-send-email-Matthieu.Moy@imag.fr>

```
On Wed, Jul 29, 2015 at 9:22 PM, Matthieu Moy <Matthieu.Moy@imag.fr> wrote:
> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
> ---
> This is meant to be applied on top of kn/for-each-ref.
>
>  ref-filter.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/ref-filter.c b/ref-filter.c
> index 43502a4..3fbbbeb 100644
> --- a/ref-filter.c
> +++ b/ref-filter.c
> @@ -868,8 +868,8 @@ static int ref_filter_handler(const char *refname, const struct object_id *oid,
>         struct ref_array_item *ref;
>
>         if (flag & REF_BAD_NAME) {
> -                 warning("ignoring ref with broken name %s", refname);
> -                 return 0;
> +               warning("ignoring ref with broken name %s", refname);
> +               return 0;
>         }
>
>         if (*filter->name_patterns && !match_name_as_path(filter->name_patterns, refname))
> --
> 2.5.0.rc0.7.ge1edd74.dirty
>
> --
> To unsubscribe from this list: send the line "unsubscribe git" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Thanks for this :)

-- 
Regards,
Karthik Nayak

```

## Junio C Hamano, 2015-07-29 17:49

Subject: Re: [PATCH] ref-filter: fix indentation
Message-ID: <xmqq3806am76.fsf@gitster.dls.corp.google.com>
URL: https://gitlist.dev/e/xmqq3806am76.fsf%40gitster.dls.corp.google.com
In-Reply-To: <1438185150-31730-1-git-send-email-Matthieu.Moy@imag.fr>

```
Matthieu Moy <Matthieu.Moy@imag.fr> writes:

> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
> ---
> This is meant to be applied on top of kn/for-each-ref.

Hmm, as 2.5 is already out and we can rewind 'next' in the coming
couple of days, I am tempted to squash this in instead of applying
it on top.

Thanks.

>  ref-filter.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/ref-filter.c b/ref-filter.c
> index 43502a4..3fbbbeb 100644
> --- a/ref-filter.c
> +++ b/ref-filter.c
> @@ -868,8 +868,8 @@ static int ref_filter_handler(const char *refname, const struct object_id *oid,
>  	struct ref_array_item *ref;
>  
>  	if (flag & REF_BAD_NAME) {
> -		  warning("ignoring ref with broken name %s", refname);
> -		  return 0;
> +		warning("ignoring ref with broken name %s", refname);
> +		return 0;
>  	}
>  
>  	if (*filter->name_patterns && !match_name_as_path(filter->name_patterns, refname))

```

## Matthieu Moy, 2015-07-29 21:18

Subject: Re: [PATCH] ref-filter: fix indentation
Message-ID: <vpqmvyeek8n.fsf@anie.imag.fr>
URL: https://gitlist.dev/e/vpqmvyeek8n.fsf%40anie.imag.fr
In-Reply-To: <xmqq3806am76.fsf@gitster.dls.corp.google.com>

```
Junio C Hamano <gitster@pobox.com> writes:

> Matthieu Moy <Matthieu.Moy@imag.fr> writes:
>
>> Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr>
>> ---
>> This is meant to be applied on top of kn/for-each-ref.
>
> Hmm, as 2.5 is already out and we can rewind 'next' in the coming
> couple of days, I am tempted to squash this in instead of applying
> it on top.

Perfect.

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/

```
