Re: [PATCH v3 0/3] handle multiline in-body headers

2016-09-21 Thread Junio C Hamano
Jonathan Tan writes: > With the above change, it is actually no longer necessary to make > is_scissors_line take plain char * (the second patch) - I think that > that patch still improves the code, but let me know if you want me to > remove it from this patch set. I agree with you that it is an

Re: [PATCH v3 0/3] handle multiline in-body headers

2016-09-20 Thread Jeff King
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 si

[PATCH v3 0/3] handle multiline in-body headers

2016-09-20 Thread Jonathan Tan
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) With the above change, it is actually no