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.

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_endian: remove the unused Curl_write64_le function #7280

Closed
wants to merge 1 commit into from
Closed

curl_endian: remove the unused Curl_write64_le function #7280

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jun 19, 2021

The last usage was removed in cca455a

@bagder bagder added the tidy-up label Jun 19, 2021
@monnerat
Copy link
Contributor

Approved, as I've noticed it became unused after cca455a, but decided to keep it for an eventual future use,

@bagder
Copy link
Member Author

bagder commented Jun 20, 2021

If we ever need it in the future, we can bring it back.

@bagder bagder closed this in 4331c6d Jun 20, 2021
@bagder bagder deleted the bagder/write64-remove branch June 20, 2021 21:39
ryandesign added a commit to ryandesign/curl that referenced this pull request Nov 22, 2022
@ryandesign
curl_endian: remove Curl_write64_le from header
2778505 
The actual function was already removed in 4331c6d.

See curl#7280
@ryandesign ryandesign mentioned this pull request Nov 22, 2022
Closed
bagder pushed a commit that referenced this pull request Nov 23, 2022
@ryandesign @bagder
curl_endian: remove Curl_write64_le from header
e007ace 
The actual function was already removed in 4331c6d.

See #7280
Closes #9968
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
tidy-up
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@bagder @monnerat