Skip to content
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.

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

Added UTF-8 checking. #107

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Added UTF-8 checking. #107

merged 1 commit into from
Dec 23, 2023

Conversation

redboltz
Copy link
Owner

No description provided.

@codecov-commenter
Copy link

Codecov Report

Attention: 25 lines in your changes are missing coverage. Please review.

Comparison is base (798e2d8) 89.04% compared to head (5977b30) 88.88%.
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
- Coverage   89.04%   88.88%   -0.16%     
==========================================
  Files          94       95       +1     
  Lines       10187    10249      +62     
==========================================
+ Hits         9071     9110      +39     
- Misses       1116     1139      +23     
Files Coverage Δ
include/async_mqtt/util/utf8validate.hpp 100.00% <100.00%> (ø)
include/async_mqtt/packet/property.hpp 97.40% <50.00%> (-0.28%) ⬇️
include/async_mqtt/packet/v3_1_1_publish.hpp 90.07% <50.00%> (-1.27%) ⬇️
include/async_mqtt/packet/v3_1_1_subscribe.hpp 85.24% <50.00%> (-1.20%) ⬇️
include/async_mqtt/packet/v3_1_1_unsubscribe.hpp 87.61% <50.00%> (-1.49%) ⬇️
include/async_mqtt/packet/v5_publish.hpp 93.79% <50.00%> (-0.69%) ⬇️
include/async_mqtt/packet/v5_subscribe.hpp 86.12% <50.00%> (-0.86%) ⬇️
include/async_mqtt/packet/v5_unsubscribe.hpp 88.19% <50.00%> (-1.10%) ⬇️
include/async_mqtt/packet/v3_1_1_connect.hpp 84.33% <50.00%> (-2.01%) ⬇️
include/async_mqtt/packet/v5_connect.hpp 84.85% <50.00%> (-1.54%) ⬇️

... and 1 file with indirect coverage changes

@redboltz redboltz merged commit c7a5326 into main Dec 23, 2023
11 checks passed
@redboltz redboltz deleted the add_utf8_check branch December 23, 2023 15:04
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.

2 participants