Re: [PATCH 2/1] mailinfo: don't insert header prefix for handle_content_type()

From: Junio C Hamano <gitster@pobox.com>
Date: Feb 10, 2020, 17:27 UTC
Message-ID: <xmqqlfpa1hg5.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <35ef5149-2da9-a147-fb5b-8f175ff617e7@web.de>

René Scharfe <l.s.r@web.de> writes:

Show 8 quoted lines

> handle_content_type() only cares about the value after "Content-Type: ";
> there is no need to insert that string for it.
>
> Suggested-by: Eric Sunshine <sunshine@sunshineco.com>
> Signed-off-by: René Scharfe <l.s.r@web.de>
> ---
>  mailinfo.c | 1 -
>  1 file changed, 1 deletion(-)

OK. There is only one caller that just saw that the line it is looking at begins with "Content-Type:", and the only thing the helper function does is to look for a handful of attr=value pairs (without even looking at the primary thing the header wants to convey---the type itself, like text/plain etc.). The helper may want to learn a bit more careful than the current implementation in doing what it does, but it does not need to see the header name to do so. It's not like we will be accepting more than one kind of content-type like header with this helper and make it capable of tailoring its behaviour based on which exact header type it is.

Will queue.  Thanks.

Show 14 quoted lines

> diff --git a/mailinfo.c b/mailinfo.c
> index 543962d40c..402ef04dd1 100644
> --- a/mailinfo.c
> +++ b/mailinfo.c
> @@ -570,7 +570,6 @@ static int check_header(struct mailinfo *mi,
>  		len = strlen("Content-Type: ");
>  		strbuf_add(&sb, line->buf + len, line->len - len);
>  		decode_header(mi, &sb);
> -		strbuf_insertstr(&sb, 0, "Content-Type: ");
>  		handle_content_type(mi, &sb);
>  		ret = 1;
>  		goto check_header_out;
> --
> 2.25.0