Refactor fields_base#149
Conversation
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
90cacc5 to
155484b
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
155484b to
2bf9980
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
2bf9980 to
e6867ca
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #149 +/- ##
===========================================
+ Coverage 91.56% 91.82% +0.26%
===========================================
Files 76 76
Lines 5001 5005 +4
===========================================
+ Hits 4579 4596 +17
+ Misses 422 409 -13
... and 2 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
| BOOST_ASSERT(id != field::unknown); | ||
| // Precondition violation | ||
| if(id == field::unknown) | ||
| detail::throw_logic_error(); |
There was a problem hiding this comment.
we store unknown fields using the id field::unknown. calling erase(id) or set(id, value) with id == field::unknown would affect all these fields, which is not what users expect.
I made it an exception because it's a precondition violation that's likely to occur in user code.
for example, this can happen when generating a response based on the field id of a received request. if the field is not well-known the parser assigns it the field::unknown id.
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
98b4527 to
3ad3377
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
1 similar comment
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
2b7a82a to
caa0963
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
caa0963 to
0773ca1
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
0773ca1 to
169c7c2
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
169c7c2 to
2e53b76
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
2e53b76 to
17f5bb3
Compare
|
An automated preview of the documentation is available at https://149.http-proto.prtest.cppalliance.org/index.html |
No description provided.