New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CURLOPT_READFUNCTION doc should be improved #3885
Comments
The EXAMPLE section links to https://curl.haxx.se/libcurl/c/ftpupload.html is that not sufficient? |
That option is used by numerous examples, for example ftpuploadfrommem. |
Apart from that little fix I don't see any actionable item here. @RoyBellingan can you elaborate on what specifically the existing documentation for this option doesn't cover? |
But that goes for all examples! They are after all examples, we don't expect any of them to be used exactly verbatim by people. They're there to show how the APIs are used and that example does show how the API is used even if the exact implementation possibly won't be used in applications. |
I agree. Examples are just that, examples. |
It would be nice to have a reference function implementation that shows how should be used, like https://curl.haxx.se/libcurl/c/getinmemory.html for https://curl.haxx.se/libcurl/c/CURLOPT_WRITEFUNCTION.html
In addition the page https://curl.haxx.se/libcurl/c/CURLOPT_UPLOAD_BUFFERSIZE.html has a link to https://curl.haxx.se/libcurl/c/CURLOPT_READFUNCTION.html but not the opposite.
The text was updated successfully, but these errors were encountered: