On 01/22/2013 10:43 AM, Marcel Böhme wrote:
>
> Dear all,
>
> There is another bug that sneaked into the speed patch of seq on 13.09.12:
>
> 560   if (all_digits_p (argv[optind])
> 561       && (n_args == 1 || all_digits_p (argv[optind + 1]))
> 562       && (n_args < 3 || STREQ ("1", argv[optind + 2]))
> 563       && !equal_width && !format_str && strlen (separator) == 1)
>
> That should probably be:
> 560   if (all_digits_p (argv[optind])
> 561       && (n_args == 1 || all_digits_p (argv[optind + 1]))
> 562       && (n_args < 3 || STREQ ("1", argv[optind + 1]))
> 563       && !equal_width && !format_str && strlen (separator) == 1)

Sigh we really messed up seq in that release :(
The attached should fix it:

I also notice another regression,
which I'll fix in a moment.

  $ seq 0 1 0
  0
  1

thanks,
Pádraig.