Skip to content

/ curl

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

lib: Make Curl_gethostname accept a const pointer #5866

Closed
wants to merge 1 commit into from
Closed

lib: Make Curl_gethostname accept a const pointer #5866

wants to merge 1 commit into from

Conversation

@emilengler
Copy link
Contributor

emilengler commented Aug 26, 2020

The address of that variable never gets changed, only the data in it so
why not make it a "char * const"?
@emilengler
lib: Make Curl_gethostname accept a const pointer
Verified
This commit was signed with a verified signature.
emilengler Emil Engler
GPG key ID: 2F6D4145C55FC7C7 Learn about signing commits
Loading status checks…
e3fc954 
The address of that variable never gets changed, only the data in it so
why not make it a "char * const"?

Closes #5866
@emilengler emilengler force-pushed the emilengler:2020-08-gethostname-name-const-pointer branch from 652fa19 to e3fc954 Aug 26, 2020
@bagder
bagder approved these changes Aug 26, 2020
View changes
@bagder
Copy link
Member

bagder commented Aug 27, 2020

Thanks!
@bagder bagder closed this in 616c30a Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagder bagder

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
@emilengler @bagder
You can’t perform that action at this time.