Re: [PATCH v10 9/9] convert: add round trip check based on 'core.checkRoundtripEncoding'
Show 10 quoted lines
> On 07 Mar 2018, at 23:52, Junio C Hamano <gitster@pobox.com> wrote:
>
> Lars Schneider <larsxschneider@gmail.com> writes:
>
>> I don't think HT makes too much sense. However, isspace() is nice
>> and I will use it. Being more permissive on the inputs should hurt.
>
> You are being incoherent in these three sentences. If you want to
> be more strict and only allow SP, then isspace() is already too
> broad, as it does allow HT (and even CR and LF).
I meant "Being more permissive on the inputs shouldN'T hurt." :-)
> I do think HT makes just as much sense as SP, though, so if you use
> isspace(), that would be perfectly fine.
Lars Schneider <larsxschneider@gmail.com>