bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
jida...@jidanni.org wrote: > The documentation doesn't say that one can also use hex args: > $ time /bin/sleep 0x10 > real0m16.007s > However not octal args: > $ time /bin/sleep 010 > real0m10.003s Interesting. Thanks for the report. That's an artifact of GNU sleep using strtod, which mea

bug#7878: "sort" bug--inconsistent single-column sorting influenced by other columns?

2011-01-21 Thread Bob Proulx
Hi Randall, Randall Lewis wrote: > Wow! So, a couple comments about how I seem to have figured out > every wrong way to use "sort" when also using "join." You did have an impressive number of cases examined! > Who would've thought that > > sort -k1 test1.txt > > would default to sort on the en

bug#7878: "sort" bug--inconsistent single-column sorting influenced by other columns?

2011-01-21 Thread Randall Lewis
Thanks Paul. Yes, it would seem that the true solution to my problem is doing the following (as you suggested): use "sort -k 1,1 -t '|'" This ensures that I sort on the first field--whereas "sort -k1 -t '|'" does not, as much as I wanted it to. ;) Since I was joining on only the first field I

bug#7878: "sort" bug--inconsistent single-column sorting influenced by other columns?

2011-01-21 Thread Paul Eggert
On 01/20/2011 11:29 PM, Randall Lewis wrote: > Also, who would've thought that the default "sort" would be incompatible with > "join" and that you would need to write the command like this every time you > wanted to use "join"? > > LC_ALL=C sort test1.txt No, "sort" and "join" use the same coll

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Pádraig Brady
On 21/01/11 09:24, Jim Meyering wrote: > +return false; > + return ISDIGIT (*p) || *p == '.'; > +} looks good. Perhaps rename to c_strtod_pre_filter() as I initially wondered why you hardcoded '.' cheers, Pádraig.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Voelker, Bernhard
Jim Meyering wrote: > That's an artifact of GNU sleep using strtod, which means "inf" and > "INFINITY" are also accepted: > > $ timeout 1 sleep inf > [Exit 124] what's wrong with `sleep inf`? Have a nice day, Berny

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
Pádraig Brady wrote: > On 21/01/11 09:24, Jim Meyering wrote: >> +return false; >> + return ISDIGIT (*p) || *p == '.'; >> +} > > looks good. Perhaps rename to c_strtod_pre_filter() > as I initially wondered why you hardcoded '.' Good idea. Done. Thanks for the quick review.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
Voelker, Bernhard wrote: > Jim Meyering wrote: >> That's an artifact of GNU sleep using strtod, which means "inf" and >> "INFINITY" are also accepted: >> >> $ timeout 1 sleep inf >> [Exit 124] > > what's wrong with `sleep inf`? Hi Volker, There's nothing terribly _wrong_ with it, but I am

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread jidanni
You see I was trying to make very special arguments to sleep so that I can make sure to kill the one I want pkill -u jidanni -fx sleep\ 22 It is not clear if that will still work. Maybe the man/info page should say a regexp that matches all valid args.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
jida...@jidanni.org wrote: > You see I was trying to make very special arguments to sleep so that I > can make sure to kill the one I want > pkill -u jidanni -fx sleep\ 22 > It is not clear if that will still work. With today's change using "sleep 22" will still act just like "sleep 22

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
jida...@jidanni.org wrote: > You see I was trying to make very special arguments to sleep so that I > can make sure to kill the one I want > pkill -u jidanni -fx sleep\ 22 > It is not clear if that will still work. Maybe the man/info page should > say a regexp that matches all valid args.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Pádraig Brady
On 21/01/11 12:48, jida...@jidanni.org wrote: > You see I was trying to make very special arguments to sleep so that I > can make sure to kill the one I want > pkill -u jidanni -fx sleep\ 22 > It is not clear if that will still work. Maybe the man/info page should > say a regexp that matche

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Eric Blake
On 01/21/2011 06:03 AM, Jim Meyering wrote: > jida...@jidanni.org wrote: >> You see I was trying to make very special arguments to sleep so that I >> can make sure to kill the one I want >> pkill -u jidanni -fx sleep\ 22 >> It is not clear if that will still work. > > With today's change u

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
Eric Blake wrote: > On 01/21/2011 06:03 AM, Jim Meyering wrote: >> jida...@jidanni.org wrote: >>> You see I was trying to make very special arguments to sleep so that I >>> can make sure to kill the one I want >>> pkill -u jidanni -fx sleep\ 22 >>> It is not clear if that will still work.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Eric Blake
On 01/21/2011 07:57 AM, Jim Meyering wrote: >> Floating point literals include hex-floats, such as 0x1p1. Does today's >> added pre-filtering to reject 0x1 accidentally also reject hex-float >> literals? (It's a bit of a shame that when hex-floats were added that >> strtod() was altered to accept

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Paul Eggert
On 01/21/2011 01:24 AM, Jim Meyering wrote: > My first reflex was to make sleep reject args like 0x... and inf. My reflex was just the opposite: why reject a notation that might be useful? Several other programs (printf, seq, sort, tail, plus many other GNU programs) also use strtod or strtold.

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Jim Meyering
Paul Eggert wrote: > On 01/21/2011 01:24 AM, Jim Meyering wrote: >> My first reflex was to make sleep reject args like 0x... and inf. > > My reflex was just the opposite: why reject a notation > that might be useful? I see that at least freebsd's /bin/sleep and the one from sunos 5.11 work just li

bug#7877: sleep takes undocumented hex args

2011-01-21 Thread Pádraig Brady
On 21/01/11 19:28, Jim Meyering wrote: > Paul Eggert wrote: >> On 01/21/2011 01:24 AM, Jim Meyering wrote: >>> My first reflex was to make sleep reject args like 0x... and inf. >> >> My reflex was just the opposite: why reject a notation >> that might be useful? > > I see that at least freebsd's /