Skip to content

socket: Remove redundant call to getsockname - #13655

Closed
orgads wants to merge 1 commit into
curl:masterfrom
orgads:no-getsockname
Closed

socket: Remove redundant call to getsockname#13655
orgads wants to merge 1 commit into
curl:masterfrom
orgads:no-getsockname

Conversation

@orgads

@orgads orgads commented May 15, 2024

Copy link
Copy Markdown
Contributor

The result "add" is unused.

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice find!

@bagder

bagder commented May 15, 2024

Copy link
Copy Markdown
Member

The broken nghttp2 CI jobs were fixed in #13656

@orgads

orgads commented May 15, 2024

Copy link
Copy Markdown
Contributor Author

The broken nghttp2 CI jobs were fixed in #13656

Thanks, rebased.

@bagder bagder closed this in 17fbed2 May 15, 2024
@bagder

bagder commented May 15, 2024

Copy link
Copy Markdown
Member

Thanks!

@orgads
orgads deleted the no-getsockname branch May 15, 2024 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants