Ok so here is a link to my branch: https://github.com/s-3ntinel/curl/tree/fix-bug-%232161
And to answer your question, why should be (total percentage) removed, now the total field is always full size of a file, so percentage would be always at 100%, so now total percentage role falls to received percentage role, if I understand it correctly. And since I'm new here, you can comment if I am doing something wrong i tried to read developer guidelines on the official site, but it's so extensive. 2018-02-22 23:14 GMT+01:00 Ray Satiro <[email protected]>: > On 2/22/2018 1:02 PM, Maroš Priputen wrote: > > Hi there, I\m new to curl (first time contributing to open source > > project also). > > > > I am working on resolving this issue ( Github: #2161 ) > > > > So far I think i fixed: Total (and) % Received > > > > ------------------------------------------------------------ > --------------------------- > > Here is a snippet (I did it like this because I didn't want to post > > images in attachments) > > > > ** Resuming transfer from byte position 1186172928 > > Total % Received % Xferd Average Speed Time Time > > Time Current > > Dload Upload > > Total Spent Left Speed > > 1432M 79 1141M 0 0 2321k 0 0:02:13 > > 0:00:04 0:02:09 2321k > > > > ------------------------------------------------------------ > --------------------------- > > > > My question is, should I leave (Time Spent), to count from beginning > > (0:00:00) every time or is it better > > if it continues, from where I left off? > > > > Btw, i also removed (% Total) because it doesn't make sense in this > > format. > > > > And since I'm new here it it appropriate to post these kind of stuff > > here or should I do this on github? > > > > Thanks. > > That's great you are working on it. I suggest participating in github > issue #2161 [1] since it is already tracked there. I think only Total > and Received size amounts should be changed, and not times. Otherwise we > would have to record the cumulative times of transfers. As far as > removing % total I don't know why that is necessary. Please comment on > the issue and share your thoughts and a link to the branch or your code. > > [1]: https://github.com/curl/curl/issues/2161 > > > ------------------------------------------------------------------- > Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library > Etiquette: https://curl.haxx.se/mail/etiquette.html
------------------------------------------------------------------- Unsubscribe: https://cool.haxx.se/list/listinfo/curl-library Etiquette: https://curl.haxx.se/mail/etiquette.html
