The following construct in progress.c core dumps on sgi:
375 dlpercen = (long)(100*(data->progress.downloaded/100) /
376 (data->progress.size_dl/100));

print data->progress.size_dl
$7 = 6
(gdb) print data->progress.downloaded
$8 = 6

as the above gives 0/0 which is a trap condition on sgi.
There's a potential similar problem just a few lines above, for the upload
speed calculation.
I know there's been a lot of changes forth and back for the progress handling,
but wouldn't it be both safer and simpler to just use floating point wherever
division is involved?

-Tor
Received on 2004-05-21