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 <p...@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.
>
>
_______________________________________________
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to