[debbugs-tracker] bug#17800: closed (seq weirdness with -0)

[GNU bug Tracking System]

Your message dated Wed, 18 Jun 2014 14:39:03 +0100
with message-id <address@hidden>
and subject line Re: bug#17800: seq weirdness with -0
has caused the debbugs.gnu.org bug report #17800,
regarding seq weirdness with -0
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
17800: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=17800
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> seq weirdness with -0
>
>
>
>
> Date: 
>
> Wed, 18 Jun 2014 11:41:43 +0200
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)
>
>
>
>
> $ ./seq --version | head -1
> seq (GNU coreutils) 8.22.119-8a51b
>
> ./seq -0 n works fine when n is a single digit:
>
> $ ./seq --separator=, -0 5
> -0,1,2,3,4,5
>
> But something weird happens when one uses a number >= 10:
>
> $ ./seq --separator=, -0 10
> -0,-1,-2,-3,-4,-5,-6,-7,-8,-9,.0,.1,.2,.3,.4,.5,.6,.7,.8,.9,/0,/1,/2,/3,/4,/5,/6,/7,/8,/9,00,01,02,03,04,05,06,07,08,09,10
>
> [It also happens without the --separator; I just use that to save
> vertical space.] This smells of ASCII, and looking at the code, the
> problem is very likely to be the seq_fast/incr functions. I don't know
> what the simplest fix is, though.
>
> Rasmus
>
>
>
>
> --- End Message ---

--- Begin Message --- Subject: Re: bug#17800: seq weirdness with -0 Date: Wed, 18 Jun 2014 14:39:03 +0100 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2

On 06/18/2014 01:03 PM, Pádraig Brady wrote:
> On 06/18/2014 10:41 AM, Rasmus Villemoes wrote:
>> $ ./seq --version | head -1
>> seq (GNU coreutils) 8.22.119-8a51b
>>
>> ./seq -0 n works fine when n is a single digit:
>>
>> $ ./seq --separator=, -0 5
>> -0,1,2,3,4,5
>>
>> But something weird happens when one uses a number >= 10:
>>
>> $ ./seq --separator=, -0 10
>> -0,-1,-2,-3,-4,-5,-6,-7,-8,-9,.0,.1,.2,.3,.4,.5,.6,.7,.8,.9,/0,/1,/2,/3,/4,/5,/6,/7,/8,/9,00,01,02,03,04,05,06,07,08,09,10
>>
>> [It also happens without the --separator; I just use that to save
>> vertical space.] This smells of ASCII, and looking at the code, the
>> problem is very likely to be the seq_fast/incr functions. I don't know
>> what the simplest fix is, though.
> 
> Ouch. I see the issue. Fix on the way...

Fast path avoidance logic updated in the attached.

thanks!
Pádraig.

seq-0.patch
Description: Text Data

--- End Message ---