Date: Thu, 1 Dec 2016 10:58:13 -0600 From: Eric van Gyzen <vangyzen@FreeBSD.org> To: Bruce Evans <brde@optusnet.com.au> Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r309331 - head/usr.bin/locale Message-ID: <c12f57a1-137d-dac9-e824-ef3baf6eb312@FreeBSD.org> In-Reply-To: <20161201082459.T1285@besplex.bde.org> References: <201611301834.uAUIYfQs075427@repo.freebsd.org> <20161201082459.T1285@besplex.bde.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 11/30/2016 18:19, Bruce Evans wrote: > On Wed, 30 Nov 2016, Eric van Gyzen wrote: > >> Log: >> Include limits.h for CHAR_MAX >> >> This was needed on stable/10. Apparently, sys/param.h supplies CHAR_MAX >> on head. Include limits.h anyway, for consistency, and because C >> says so. > > sys/param.actually supplies CHAR_MAX (undocumented namespace pollution) in > all versions of FreeBSD. Yes, I was too terse. I meant, locale.c gets CHAR_MAX from sys/param.h on head (and 11), but does not include sys/param.h on 10. > Untested fixes and cleanups: > > Y diff -u2 locale.c~ locale.c Thanks. Committed. Eric
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?c12f57a1-137d-dac9-e824-ef3baf6eb312>