build: Fix compilation errors for boost 1.9#754
Conversation
7d3aa46 to
aeaf68a
Compare
Summary of ChangesHello @uristdwarf, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request primarily focuses on updating the project to be compatible with Boost 1.9, resolving compilation errors, and enhancing the robustness of timer mechanisms. The changes involve migrating Boost.Asio timers from system-time-based Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request addresses compilation errors with Boost 1.9 and updates timer implementations to use steady_timer instead of deadline_timer and expires_after with std::chrono::milliseconds instead of expires_from_now with boost::posix_time::millisec. This change improves the robustness of the timing mechanisms by switching from system time to monotonic time, which is less susceptible to system clock adjustments. Additionally, boost::shared_ptr.hpp and boost/chrono.hpp headers are included where needed, and the Boost system component is removed from find_package in cmake/Libraries.cmake as it's no longer explicitly required by the updated timer usage. All original comments have been retained as they are valid and do not contradict any provided rules.
ralcolea
left a comment
There was a problem hiding this comment.
Great job @uristdwarf! 👍
I also tested the Molecule suite with this branch and all the tests passed successfully!
This also changes from posix_timer (which I think is system time) to steady_timer (monotonic time), which should help avoid bugs with system time. Signed-off-by: Urmas Rist <urmas@leil.io>
aeaf68a to
e15cf94
Compare
This also changes from posix_timer (which I think is system time) to steady_timer (monotonic time), which should help avoid bugs with system time.