svn commit: r311109 - head/usr.bin/patch

[Conrad Meyer]

I was suggesting using UINT32_MAX/2 on all platforms (which is safe
everywhere).

Best,
Conrad

On Mon, Jan 2, 2017 at 11:48 AM Pedro Giffuni <pfg at freebsd.org> wrote:

> Hi;
>
>
>
> On 01/02/17 13:28, Conrad Meyer wrote:
>
> > IMO this patch was mostly fine, you just need to restrict line length
>
> > to UINT32_MAX/2 instead of UINT_MAX (== UINT32_MAX) for 32-bit
>
> > platforms.
>
> >
>
> > Best,
>
> > Conrad
>
> >
>
>
>
> I dislike the idea of special cases for some platforms. I guess moving
>
> to int/INT_MAX would be the way to go.
>
>
>
> TBH, no one has asked for patches wider than USHRT_MAX though.
>
>
>
> Pedro.
>
>