Skip to content

MultipartParser: use Pull.done instead of Pull.pure. #6449

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
Jun 5, 2022
Merged

MultipartParser: use Pull.done instead of Pull.pure. #6449

merged 1 commit into from
Jun 5, 2022

Conversation

diesalbla
Copy link
Contributor

No description provided.

@diesalbla
MultipartParser: use Pull.done instead of Pull.pure.
8ef47d2 
It is already a constant in fs2, so saves two allocs per call.
@mergify mergify bot added series/0.23 PRs targeting 0.23.x module:core labels Jun 5, 2022
@armanbilge armanbilge added the behind-the-scenes Appreciated, but not user-facing label Jun 5, 2022
@armanbilge armanbilge merged commit a731d95 into http4s:series/0.23 Jun 5, 2022
@diesalbla diesalbla deleted the micropt_use_pull_done branch June 5, 2022 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@armanbilge armanbilge armanbilge approved these changes

Assignees
No one assigned
Labels
behind-the-scenes Appreciated, but not user-facing module:core series/0.23 PRs targeting 0.23.x
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@diesalbla @armanbilge