Re: svn commit: r227369 - head/bin/sh

2011-11-23 Thread Bruce Evans
On Tue, 22 Nov 2011, Jilles Tjoelker wrote: On Mon, Nov 21, 2011 at 06:29:06PM +1100, Bruce Evans wrote: On Wed, 9 Nov 2011, Jilles Tjoelker wrote: On Wed, Nov 09, 2011 at 09:35:51AM +0100, Stefan Farfeleder wrote: Isn't the behaviour undefined too when you convert an out-of-range uintmax_t

Re: svn commit: r227369 - head/bin/sh

2011-11-21 Thread Jilles Tjoelker
On Mon, Nov 21, 2011 at 06:29:06PM +1100, Bruce Evans wrote: On Wed, 9 Nov 2011, Jilles Tjoelker wrote: On Wed, Nov 09, 2011 at 09:35:51AM +0100, Stefan Farfeleder wrote: Isn't the behaviour undefined too when you convert an out-of-range uintmax_t value back into an intmax_t value? The

Re: svn commit: r227369 - head/bin/sh

2011-11-20 Thread Bruce Evans
On Wed, 9 Nov 2011, Jilles Tjoelker wrote: On Wed, Nov 09, 2011 at 09:35:51AM +0100, Stefan Farfeleder wrote: On Tue, Nov 08, 2011 at 11:54:39PM +, Jilles Tjoelker wrote: Author: jilles Date: Tue Nov 8 23:54:39 2011 New Revision: 227369 URL: http://svn.freebsd.org/changeset/base/227369

Re: svn commit: r227369 - head/bin/sh

2011-11-09 Thread Stefan Farfeleder
On Tue, Nov 08, 2011 at 11:54:39PM +, Jilles Tjoelker wrote: Author: jilles Date: Tue Nov 8 23:54:39 2011 New Revision: 227369 URL: http://svn.freebsd.org/changeset/base/227369 Log: sh: Remove undefined behaviour due to overflow in +/-/* in arithmetic. With i386 base gcc and

Re: svn commit: r227369 - head/bin/sh

2011-11-09 Thread Jilles Tjoelker
On Wed, Nov 09, 2011 at 09:35:51AM +0100, Stefan Farfeleder wrote: On Tue, Nov 08, 2011 at 11:54:39PM +, Jilles Tjoelker wrote: Author: jilles Date: Tue Nov 8 23:54:39 2011 New Revision: 227369 URL: http://svn.freebsd.org/changeset/base/227369 Log: sh: Remove undefined

svn commit: r227369 - head/bin/sh

2011-11-08 Thread Jilles Tjoelker
Author: jilles Date: Tue Nov 8 23:54:39 2011 New Revision: 227369 URL: http://svn.freebsd.org/changeset/base/227369 Log: sh: Remove undefined behaviour due to overflow in +/-/* in arithmetic. With i386 base gcc and i386 base clang, arith_yacc.o remains unchanged. Modified: