First, expanded tabs to four spaces in the main module. It was using spaces in some places, and tabs in other, so, preferred to keep spaces (four). Any way, we can remove that commit if you like.
Other than that, tested if we can send the curl object to the callback function (yes we can) and if we can obtain the current download speed. Also, tested the content type is available and has a sensible value.
And only now, when writing about the possibility to add a default of sending the curl object in case no progress data is set, I find out that is already implemented. Damn, an easy and interesting patch I could write, and it is not needed :smile:
Hey
First, expanded tabs to four spaces in the main module. It was using spaces in some places, and tabs in other, so, preferred to keep spaces (four). Any way, we can remove that commit if you like.
Other than that, tested if we can send the curl object to the callback function (yes we can) and if we can obtain the current download speed. Also, tested the content type is available and has a sensible value.
And only now, when writing about the possibility to add a default of sending the curl object in case no progress data is set, I find out that is already implemented. Damn, an easy and interesting patch I could write, and it is not needed :smile: