Skip to content

Syncer mutex to fix single_consumer_queue::peek() issues - #9309

Merged
maloel merged 4 commits into
realsenseai:developmentfrom
maloel:peek
Jul 1, 2021
Merged

Syncer mutex to fix single_consumer_queue::peek() issues#9309
maloel merged 4 commits into
realsenseai:developmentfrom
maloel:peek

Conversation

@maloel

@maloel maloel commented Jun 29, 2021

Copy link
Copy Markdown
Contributor

And revised peek() mechanism, too...

Tracked on [LRS-162]

* add mutex to syncer_process_unit::stop()
@maloel
maloel requested a review from Nir-Az June 29, 2021 09:16
Comment thread src/sync.cpp
Nir-Az
Nir-Az approved these changes Jun 30, 2021

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Check about the !

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@maloel maloel changed the title Rework single_consumer_queue::peek() to be more thread-safe Syncer mutex to fix single_consumer_queue::peek() issues Jul 1, 2021
@maloel
maloel merged commit 8cdfb20 into realsenseai:development Jul 1, 2021
@maloel
maloel deleted the peek branch December 22, 2021 07:37
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