Skip to content

perf(util): optimize <BufList as Buf>::has_remaining#94

Merged
seanmonstar merged 1 commit intohyperium:masterfrom
tesaguri:buf-list-has-remaining
Feb 21, 2024
Merged

perf(util): optimize <BufList as Buf>::has_remaining#94
seanmonstar merged 1 commit intohyperium:masterfrom
tesaguri:buf-list-has-remaining

Conversation

@tesaguri
Copy link
Copy Markdown
Contributor

This allows short-circuiting when there is a non-empty chunk.

This allows short-circuiting when there is a non-empty chunk.
@tesaguri tesaguri force-pushed the buf-list-has-remaining branch from 10a9f72 to 22e6876 Compare September 10, 2023 12:19
@tesaguri
Copy link
Copy Markdown
Contributor Author

The CI failure doesn't seem to be relevant to this patch.

@seanmonstar seanmonstar merged commit bffa8c5 into hyperium:master Feb 21, 2024
@tesaguri tesaguri deleted the buf-list-has-remaining branch February 22, 2024 12:24
@yanns
Copy link
Copy Markdown

yanns commented Mar 11, 2024

Could we have a release for this?

@seanmonstar
Copy link
Copy Markdown
Member

I checked the benchmarks and didn't see much difference with this. But I've filed #112 which does help a lot.

@yanns
Copy link
Copy Markdown

yanns commented Mar 11, 2024

For me, that change was a big improvement: tokio-rs/axum#2381 (reply in thread)
I'll try #112

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants