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

sendf: Remove unnecessary null buffer check #9801

Closed
wants to merge 1 commit into from
Closed

sendf: Remove unnecessary null buffer check #9801

wants to merge 1 commit into from

Conversation

JDepooter
Copy link
Contributor

At this point, the psnd->buffer will always exist. We have already allocated a new buffer if one did not previously exist, and returned from the function if the allocation failed.

@JDepooter
Remove unnecessary condition
251d360 
At this point, the psnd->buffer will always exist. We have already allocated a new buffer if one did not previously exist, and returned from the function if the allocation failed.
@jay jay added the tidy-up label Oct 26, 2022
@bagder bagder closed this in df77eff Oct 26, 2022
@bagder
Copy link
Member

bagder commented Oct 26, 2022

Thanks!

@JDepooter JDepooter deleted the pre_receive_plain_unnecessary_condition branch October 26, 2022 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bagder bagder bagder approved these changes

Assignees
No one assigned
Labels
hacktoberfest-accepted tidy-up
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@JDepooter @bagder @jay