Skip to content
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.

Sign up for GitHub

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

Jump to bottom

curl_version_info.3 documentation missing "const" for protocols field #225

Closed
jmarshall opened this issue Apr 18, 2015 · 1 comment
Closed

curl_version_info.3 documentation missing "const" for protocols field #225

jmarshall opened this issue Apr 18, 2015 · 1 comment
Assignees
@bagder
Labels
documentation

Comments

@jmarshall
Copy link
Contributor

Rather minor, but… the curl_version_info.3 man page documents the curl_version_info_data struct's fields, including

const char **protocols;   /* list of protocols */

However protocols is in fact a const char * const * — which makes a difference when user code is declaring a variable to traverse the list.
bagder added a commit that referenced this issue Apr 18, 2015
@bagder
curl_version_info.3: fixed the 'protocols' variable type
63c64e0 
Reported-by: John Marshall
Bug: #225
@bagder bagder self-assigned this Apr 18, 2015
@bagder
Copy link
Member

bagder commented Apr 18, 2015

Thanks a lot, fixed now in 63c64e0

@bagder bagder closed this as completed Apr 18, 2015
jgsogo pushed a commit to jgsogo/curl that referenced this issue Oct 19, 2015
@bagder @jgsogo
curl_version_info.3: fixed the 'protocols' variable type
05faeb8 
Reported-by: John Marshall
Bug: curl#225
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2018
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bagder bagder

Labels
documentation
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jmarshall @bagder