Skip to content

reduce chunk header size based on buffer capacity#150

Merged
ashtum merged 1 commit into
cppalliance:developfrom
ashtum:optimize-chunk-header-length
Jul 28, 2025
Merged

reduce chunk header size based on buffer capacity#150
ashtum merged 1 commit into
cppalliance:developfrom
ashtum:optimize-chunk-header-length

Conversation

@ashtum
Copy link
Copy Markdown
Collaborator

@ashtum ashtum commented Jul 28, 2025

resolves #141

@cppalliance-bot
Copy link
Copy Markdown

An automated preview of the documentation is available at https://150.http-proto.prtest.cppalliance.org/index.html

@ashtum ashtum force-pushed the optimize-chunk-header-length branch from 5542d15 to 75ef14b Compare July 28, 2025 05:43
@cppalliance-bot
Copy link
Copy Markdown

An automated preview of the documentation is available at https://150.http-proto.prtest.cppalliance.org/index.html

@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 28, 2025

Codecov Report

❌ Patch coverage is 98.31933% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.93%. Comparing base (f742d7c) to head (04d6788).
⚠️ Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
src/serializer.cpp 98.31% 2 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #150      +/-   ##
===========================================
+ Coverage    91.82%   91.93%   +0.10%     
===========================================
  Files           76       76              
  Lines         5005     4974      -31     
===========================================
- Hits          4596     4573      -23     
+ Misses         409      401       -8     
Files with missing lines Coverage Δ
include/boost/http_proto/serializer.hpp 100.00% <ø> (ø)
src/serializer.cpp 92.47% <98.31%> (+1.14%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f742d7c...04d6788. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ashtum ashtum force-pushed the optimize-chunk-header-length branch from 75ef14b to 9739446 Compare July 28, 2025 08:06
@cppalliance-bot
Copy link
Copy Markdown

An automated preview of the documentation is available at https://150.http-proto.prtest.cppalliance.org/index.html

@ashtum ashtum force-pushed the optimize-chunk-header-length branch from 9739446 to d48a12d Compare July 28, 2025 08:53
@cppalliance-bot
Copy link
Copy Markdown

An automated preview of the documentation is available at https://150.http-proto.prtest.cppalliance.org/index.html

@ashtum ashtum force-pushed the optimize-chunk-header-length branch from d48a12d to 04d6788 Compare July 28, 2025 09:22
@cppalliance-bot
Copy link
Copy Markdown

An automated preview of the documentation is available at https://150.http-proto.prtest.cppalliance.org/index.html

@ashtum ashtum merged commit d90cfcf into cppalliance:develop Jul 28, 2025
61 checks passed
@ashtum ashtum deleted the optimize-chunk-header-length branch October 4, 2025 05:40
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.

serializer could reserve smaller chunk headers

2 participants