Re: [coreutils] coreutils-7.0 expr exposes long-standing bug in matlab startup script

[Jim Meyering]

Pádraig Brady <address@hidden> wrote:
> Jim Meyering wrote:
>> For reference, here's a small patch I just wrote that solves
>> at least some of the problems reported here, while retaining
>> the ability to use MP at run time.
>
> I'd essentially done the same thing, so I think that will solve
> most of the backwards compat issues.
>
> Note if leaving the --bignum options you should probably also do:
>
> -Usage: %s EXPRESSION\n\
> -  or:  %s OPTION\n\
> +Usage: %s [OPTION] EXPRESSION\n\
>
> Though I strongly agree with Paul that it shouldn't be an option.
> I was cooking up a patch as I also noticed the 64 bit int regression:
>
> $ expr.orig 4294967296 + 1
> 4294967297
> $ ./expr --no-bignum 4294967296 + 1
> ./expr: 4294967296: Numerical result out of range
>
> But he's faster than me so less testing for me to do :)
> I'll add some extra tests for the "-11 + 3" case etc.
> if I don't notice any going in in the next while.

Thanks!  Tests for this would be most welcome.