search

cccssw / xxhash

Automatically exported from code.google.com/p/xxhash
Other
0 stars 0 forks source link

Main output goes to stderr #23

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The main output of the hash and file name are output to stderr rather than 
stdout.

Original issue reported on code.google.com by zcwxod on 17 Oct 2014 at 8:37

GoogleCodeExporter commented 9 years ago 
Good point !
This will be corrected

Original comment by yann.col...@gmail.com on 17 Oct 2014 at 8:48

GoogleCodeExporter commented 9 years ago 
A tentative fix is proposed at https://github.com/Cyan4973/xxHash/tree/dev

Original comment by yann.col...@gmail.com on 18 Oct 2014 at 11:19

GoogleCodeExporter commented 9 years ago 
Using the proposed branch for a fix, xxHash outputs
"Loadin"
to stdout while computing the hash. That would be better to go to stderr to 
separate status output from the main product. Also the "g" is missing from 
"Loading".

Original comment by zcwxod on 22 Oct 2014 at 6:14

GoogleCodeExporter commented 9 years ago 
Good points. I'll correct them in next dev release.

Original comment by yann.col...@gmail.com on 22 Oct 2014 at 10:57

GoogleCodeExporter commented 9 years ago 
New patch, available at https://github.com/Cyan4973/xxHash/tree/dev

"Loading" is not missing any 'g' though. I'm suspecting it might be a display 
issue (partial line overwrite).

Original comment by yann.col...@gmail.com on 23 Oct 2014 at 8:41

GoogleCodeExporter commented 9 years ago 
I clone that repository; build ; run
./xxhsum /dev/urandom 2>/dev/null
. It outputs
Loadin

What is wrong?

Original comment by zcwxod on 24 Oct 2014 at 1:01

GoogleCodeExporter commented 9 years ago 
For some reason, I'm not getting the same results.

With latest dev branch release, it outputs nothing

With previous dev branch release, it outputs : Loading /dev/urandom...

Original comment by yann.col...@gmail.com on 24 Oct 2014 at 7:11

GoogleCodeExporter commented 9 years ago 
Fixed into r38

Original comment by yann.col...@gmail.com on 1 Nov 2014 at 8:56