Skip to content

Conversation

djchhp
Copy link
Contributor

@djchhp djchhp commented Oct 9, 2025

Describe your changes

This adds a new monitor class template to everest/lib/util implementing the monitor pattern

The monitor wraps up some data structure and give pointer like access via a handle, that ensures exclusive access.

Depending on the used mutex type additional features, like getting the handle with a timeout (std::timed_mutex) or conditional wait (std::mutex) are available.

See the included doxygen documentation and the testcases for details

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have made corresponding changes to the documentation
  • I read the contribution documentation and made sure that my changes meet its requirements

@djchhp djchhp marked this pull request as ready for review October 9, 2025 09:40
@djchhp djchhp force-pushed the feat/async_util branch 2 times, most recently from 8beaf61 to 436a1ac Compare October 9, 2025 10:16
Copy link
Member

@james-ctc james-ctc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me

@djchhp djchhp added this pull request to the merge queue Oct 15, 2025
@djchhp djchhp removed this pull request from the merge queue due to a manual request Oct 15, 2025
* This adds a new monitor class for simplified concurrency handling
* New directory and namespace structure in 'util' lib

Signed-off-by: Jan Christoph Habig <[email protected]>
@djchhp djchhp enabled auto-merge October 15, 2025 08:51
@djchhp djchhp added this pull request to the merge queue Oct 15, 2025
Merged via the queue into main with commit 4a684f6 Oct 15, 2025
11 of 12 checks passed
@djchhp djchhp deleted the feat/async_util branch October 15, 2025 09:30
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.

4 participants