--- "johnmatthews2000" <jm5...@...> wrote: > > > 9. You print out the user's average time before calling > > > statistics_append, so the printed average doesn't include the last > > > result. I don't know if this is deliberate, but it looks like a bug. > > > > It is a feature: It causes the user to compete against his previous > > performance. > > Ok - so perhaps you could change the output to make this clearer. Instead of: > > You took 9 seconds to calculate it right. > Your average was 8 seconds. > > which makes it sound like the 9 seconds is included in the average, how about: > > Your average was 8 seconds. > This time you took 9 seconds to calculate it right.
I'm thinking about: You took 9 seconds to calculate it right. This is 12% slower than your average. > > > 10. In the fopen() statement you use mode "rb+"/"wb+". Although "the b is > > > ignored on all POSIX conforming systems, including Linux" (Linux fopen > > > man page), it implies you are opening a binary file. But the statistics > > > file is a text file, so you shouldn't use 'b' here. > > > > I use a dual boot machine and I want to share the statistics file. So it > > must be compatible between architectures. > > How does using 'b' help? Have you tried it without the 'b'? Yes: If compiled with dmc and run on Windows or Wine a \r is added before each new \n . > > P.S.: The wallpaper's glue's solvent is making me crazy. P.S.: The glue's solvent gives headache. The thinner gives slowness, sleepiness and decreases the ability to think.
