Skip to content

To Be Validated: Replace Boost Dependencies Integrated to the Standard C++17 Lib#485

Draft
Ohisemega wants to merge 9 commits into
masterfrom
update_C++17_boost_dep
Draft

To Be Validated: Replace Boost Dependencies Integrated to the Standard C++17 Lib#485
Ohisemega wants to merge 9 commits into
masterfrom
update_C++17_boost_dep

Conversation

@Ohisemega

@Ohisemega Ohisemega commented Mar 23, 2026

Copy link
Copy Markdown
Contributor

Convert all boost::shared_ptr instances to std::shared_ptr.
Convert boost::enabled_shared_from_this to std::enabled_shared_from_this.
Convert boost::make_shared to std::make_shared.
Convert boost::mutex to std::mutex.
Convert boost::lock_guard to std::lock_guard.
Convert boost::unique_lock to std::unique_lock.
Convert boost::recursive_lock to std::recursive_lock.
Convert boost::condition_variable to std::condition_variable.
Convert boost::thread to std::thread.
Convert boost::scoped_ptr to std::unique_ptr.
Convert boost::any to std::any.
Convert boost::variant to std::variant.
Convert boost::placeholders to std::placeholders.
Convert boost::bind to std::bind.
Convert boost::function to std::function.
Convert boost::ref to std::ref.
Convert boost::optional to std::optional.
Convert boost::filesystem to std::filesystem.

Replace relevant boost headers with standard library headers.

Fixes #463

DO NOT MERGE YET
Open to review!

Replace all boost::shared_ptr instances with std::shared_ptr.
Replace boost::enabled_shared_from_this with std::enabled_shared_from_this.
Replace boost::make_shared with std::make_shared.

Fixes #463
@Ohisemega Ohisemega added this to the in-progress milestone Mar 23, 2026
@Ohisemega Ohisemega self-assigned this Mar 23, 2026
@Ohisemega Ohisemega added enhancement Next stage development or improvement of an existing feature or utility. C++17 This label is for all tasks related to updating Odin-data to the C++17 standard. performance This label is for all tasks related to improving, measuring and analysing Odin-data's performance labels Mar 23, 2026
@codecov

codecov Bot commented Mar 23, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 48.92473% with 95 lines in your changes missing coverage. Please review.
✅ Project coverage is 33.27%. Comparing base (bb2be94) to head (3aec880).
⚠️ Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
...pp/frameProcessor/src/FrameProcessorController.cpp 0.00% 18 Missing ⚠️
cpp/frameProcessor/src/FileWriterPlugin.cpp 32.00% 17 Missing ⚠️
cpp/frameProcessor/src/Acquisition.cpp 16.66% 10 Missing ⚠️
cpp/frameProcessor/src/FrameProcessorPlugin.cpp 33.33% 10 Missing ⚠️
cpp/frameProcessor/src/SharedMemoryController.cpp 0.00% 10 Missing ⚠️
cpp/frameReceiver/src/FrameReceiverController.cpp 0.00% 6 Missing ⚠️
cpp/frameProcessor/src/KafkaProducerPlugin.cpp 44.44% 5 Missing ⚠️
cpp/frameProcessor/src/DummyUDPProcessPlugin.cpp 0.00% 4 Missing ⚠️
cpp/common/include/ClassLoader.h 0.00% 3 Missing ⚠️
cpp/frameProcessor/src/HDF5File.cpp 70.00% 3 Missing ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #485      +/-   ##
==========================================
+ Coverage   33.24%   33.27%   +0.03%     
==========================================
  Files          94       94              
  Lines        7592     7600       +8     
  Branches      670      670              
==========================================
+ Hits         2524     2529       +5     
- Misses       4875     4878       +3     
  Partials      193      193              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Ohisemega Ohisemega changed the title boost::shared_ptr to std::shared_ptr Replace Boost Dependencies Integrated to the Standard C++17 Lib Mar 23, 2026
@Ohisemega
Ohisemega marked this pull request as draft March 23, 2026 14:57
Replace boost concurerency headers with C++ standard library headers - <thread> <mutex> <condition_variable>.
Covert boost::mutex o sd::mutex.
Convert boost::lock_guard to std::lock_guard.
Convert boost:unique_lock to std::unique_lock.
Convert boost::recursive_lock to std::recursive_lock.
Convert boost::condition_variable to std::condition_variable.
Convert boost::thread to std::thread.
Convert boost::scoped_ptr to std::unique_ptr.

    Fixes #463
Replace boost::any with std::any.
Replace boost::variant with std::variant.
Replace boost::placeholders with std::placeholders.
Replace boost::bind with std::bind.
Replace boost::function with std::function.
Replace boost::ref with std::ref.
Replace boost::optional std::optional.
Replace relevant boost headers with standard library headers.
@Ohisemega
Ohisemega marked this pull request as ready for review April 4, 2026 19:17
@Ohisemega Ohisemega added the To Be Validated PRs that should NOT be merged until validated by profiling and real-time detector testing. label Apr 7, 2026
@Ohisemega Ohisemega changed the title Replace Boost Dependencies Integrated to the Standard C++17 Lib To Be Validated: Replace Boost Dependencies Integrated to the Standard C++17 Lib Apr 7, 2026
@Ohisemega Ohisemega added Long-term Work In Progress This PR is in active, long-term development! and removed performance This label is for all tasks related to improving, measuring and analysing Odin-data's performance labels Apr 7, 2026
@Ohisemega
Ohisemega marked this pull request as draft April 7, 2026 12:59
Replace boost::filesystem with std::filesystem.
Replace boost::shared_ptr with std::shared_ptr in BloscPlugin and BloscPluginTest.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

C++17 This label is for all tasks related to updating Odin-data to the C++17 standard. enhancement Next stage development or improvement of an existing feature or utility. Long-term Work In Progress This PR is in active, long-term development! To Be Validated PRs that should NOT be merged until validated by profiling and real-time detector testing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

C++17: Replace Boost Dependencies Integrated to the Standard C++17 Lib.

2 participants