Skip to content

accept: switch 2nd param to take struct sockaddr * #630

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
Merged
merged 1 commit into from
Sep 20, 2022
Merged

accept: switch 2nd param to take struct sockaddr * #630

merged 1 commit into from
Sep 20, 2022

Conversation

G4Vi
Copy link
Collaborator

@G4Vi G4Vi commented Sep 20, 2022 •
edited
Loading

Fixes perl's Configure last arg of accept detection.

accept4 was left alone as a change was not needed.

jart
jart approved these changes Sep 20, 2022
View reviewed changes
Copy link
Owner

@jart jart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@jart jart merged commit 87708c5 into jart:master Sep 20, 2022
G4Vi added a commit to G4Vi/cosmopolitan that referenced this pull request Sep 21, 2022
@G4Vi
Change accept type to struct sockaddr * (jart#630)
a9d72c6
@G4Vi G4Vi deleted the accept_takes_sockaddr branch September 22, 2022 02:52
G4Vi added a commit to G4Vi/cosmopolitan that referenced this pull request Oct 11, 2022
@G4Vi
Change accept type to struct sockaddr * (jart#630)
cd1bca1
jart pushed a commit that referenced this pull request Oct 11, 2022
@G4Vi
Backport main branch improvements (#658)
f4ff172 
* Add sys_ prefix to unwrapped system calls
* This change also implements getlogin() and getlogin_r().
* Add getgroups and setgroups (#619)
* Fix getgroups / setgroups tests across platforms. See #619
* Change accept type to struct sockaddr * (#630)
* vista: use old strace path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jart jart jart approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@G4Vi @jart