cppalliance / http_proto

HTTP/1 parsing and serialization algorithms using C++11
https://develop.http-proto.cpp.al/
Boost Software License 1.0
23 stars 10 forks source link

serializer compression #88

Closed cmazakas closed 3 months ago

cppalliance-bot commented 3 months ago

Antora version: an automated preview of the documentation is available at https://88.http-proto.prtest.cppalliance.org/site/index.html

cppalliance-bot commented 3 months ago

GCOVR code coverage report https://88.http-proto.prtest.cppalliance.org/gcovr/index.html
LCOV code coverage report https://88.http-proto.prtest.cppalliance.org/genhtml/index.html Coverage Diff https://88.http-proto.prtest.cppalliance.org/gcovr/coverage_diff.txt

codecov[bot] commented 3 months ago

Codecov Report

Attention: Patch coverage is 93.33333% with 3 lines in your changes missing coverage. Please review.

Project coverage is 89.04%. Comparing base (041e31b) to head (90b58ca).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/cppalliance/http_proto/pull/88/graphs/tree.svg?width=650&height=150&src=pr&token=IoZbqjUnBr&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance)](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance) ```diff @@ Coverage Diff @@ ## develop #88 +/- ## =========================================== - Coverage 89.37% 89.04% -0.34% =========================================== Files 77 78 +1 Lines 4348 4435 +87 =========================================== + Hits 3886 3949 +63 - Misses 462 486 +24 ``` | [Files](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance) | Coverage Δ | | |---|---|---| | [include/boost/http\_proto/metadata.hpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=include%2Fboost%2Fhttp_proto%2Fmetadata.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-aW5jbHVkZS9ib29zdC9odHRwX3Byb3RvL21ldGFkYXRhLmhwcA==) | `100.00% <ø> (ø)` | | | [include/boost/http\_proto/serializer.hpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=include%2Fboost%2Fhttp_proto%2Fserializer.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-aW5jbHVkZS9ib29zdC9odHRwX3Byb3RvL3NlcmlhbGl6ZXIuaHBw) | `97.43% <ø> (ø)` | | | [src/detail/header.cpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=src%2Fdetail%2Fheader.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-c3JjL2RldGFpbC9oZWFkZXIuY3Bw) | `94.12% <100.00%> (+0.11%)` | :arrow_up: | | [src/rfc/transfer\_encoding\_rule.cpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=src%2Frfc%2Ftransfer_encoding_rule.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-c3JjL3JmYy90cmFuc2Zlcl9lbmNvZGluZ19ydWxlLmNwcA==) | `87.50% <100.00%> (ø)` | | | [src/serializer.cpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=src%2Fserializer.cpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-c3JjL3NlcmlhbGl6ZXIuY3Bw) | `92.03% <100.00%> (ø)` | | | [include/boost/http\_proto/detail/type\_index.hpp](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?src=pr&el=tree&filepath=include%2Fboost%2Fhttp_proto%2Fdetail%2Ftype_index.hpp&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance#diff-aW5jbHVkZS9ib29zdC9odHRwX3Byb3RvL2RldGFpbC90eXBlX2luZGV4LmhwcA==) | `89.65% <86.36%> (-10.35%)` | :arrow_down: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/cppalliance/http_proto/pull/88/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?dropdown=coverage&src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance). Last update [041e31b...90b58ca](https://app.codecov.io/gh/cppalliance/http_proto/pull/88?dropdown=coverage&src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cppalliance).
vinniefalco commented 3 months ago

Consider "serializer::stream is documented"

cppalliance-bot commented 3 months ago

Antora version: an automated preview of the documentation is available at https://88.http-proto.prtest.cppalliance.org/site/index.html

cppalliance-bot commented 3 months ago

GCOVR code coverage report https://88.http-proto.prtest.cppalliance.org/gcovr/index.html
LCOV code coverage report https://88.http-proto.prtest.cppalliance.org/genhtml/index.html Coverage Diff https://88.http-proto.prtest.cppalliance.org/gcovr/coverage_diff.txt

cppalliance-bot commented 3 months ago

Antora version: an automated preview of the documentation is available at https://88.http-proto.prtest.cppalliance.org/site/index.html

cppalliance-bot commented 3 months ago

GCOVR code coverage report https://88.http-proto.prtest.cppalliance.org/gcovr/index.html
LCOV code coverage report https://88.http-proto.prtest.cppalliance.org/genhtml/index.html Coverage Diff https://88.http-proto.prtest.cppalliance.org/gcovr/coverage_diff.txt

cppalliance-bot commented 3 months ago

Antora version: an automated preview of the documentation is available at https://88.http-proto.prtest.cppalliance.org/site/index.html

cppalliance-bot commented 3 months ago

GCOVR code coverage report https://88.http-proto.prtest.cppalliance.org/gcovr/index.html
LCOV code coverage report https://88.http-proto.prtest.cppalliance.org/genhtml/index.html Coverage Diff https://88.http-proto.prtest.cppalliance.org/gcovr/coverage_diff.txt

cppalliance-bot commented 3 months ago

Antora version: an automated preview of the documentation is available at https://88.http-proto.prtest.cppalliance.org/site/index.html

cppalliance-bot commented 3 months ago

GCOVR code coverage report https://88.http-proto.prtest.cppalliance.org/gcovr/index.html
LCOV code coverage report https://88.http-proto.prtest.cppalliance.org/genhtml/index.html Coverage Diff https://88.http-proto.prtest.cppalliance.org/gcovr/coverage_diff.txt