-
Notifications
You must be signed in to change notification settings - Fork 107
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
Resolves #222 #230
Resolves #222 #230
Conversation
More minor tweaks pertaining to #230
I have not forgotten about this. I'm just very busy recently. |
Codecov Report
@@ Coverage Diff @@
## master #230 +/- ##
=========================================
- Coverage 84.92% 84.9% -0.03%
=========================================
Files 33 33
Lines 4862 4855 -7
=========================================
- Hits 4129 4122 -7
Misses 733 733 |
test/as_buffer_async_pubsub_1.cpp
Outdated
as::buffer(*contents), | ||
false, | ||
[topic1, contents](boost::system::error_code const&) {} | ||
as::buffer("topic1"), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't want to add '\0' after the string. Please use https://github.com/redboltz/mqtt_cpp/pull/286/files#diff-c039c862d67b802e139cb035c26dcc4dR235 way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See #277 (comment)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And please do the same update other places on the test.
This code is now ready for review and merge. |
LGTM, thank you! |
Remain the deferred event that from deferred_queue during the transition process. Then, process the next deferred event. The event position is not front(), so `std::queue` concept doesn't satisfies it. I need iteration concept something like `std::deque`. In order to do that, I introduced two flags that are `defer_processing_` and `defer_again_`. In addition, to iterate deferred deque, two iterators `defer_it_` and `defer_end_` are introduced.
No description provided.