From: Jeff King <peff@peff.net>
Date: Tue, 20 Sep 2016 23:06:16 GMT
Subject: Re: [PATCH v3 0/3] handle multiline in-body headers
Message-ID: <20160920230616.n2jwhavi2dzgflno@sigill.intra.peff.net>
In-Reply-To: <cover.1474391594.git.jonathantanmy@google.com>

On Tue, Sep 20, 2016 at 10:17:50AM -0700, Jonathan Tan wrote:

> Changes since v2:
> o Removed utf8 translation before scissors line check in
>   check_inbody_header (I was thinking of support for encodings like
>   UTF-16, but I guess those don't work with the current reencode_string
>   anyway since it uses strlen internally)

Yeah, I'd be surprised if UTF-16 works very well with our code in
general. If we want to address that, though, the sanest thing is
probably to convert it internally to UTF-8 when we remove the transfer
encoding in handle_body().

-Peff