Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Queue library #25

Merged
merged 8 commits into from
Apr 14, 2024
Merged

Queue library #25

merged 8 commits into from
Apr 14, 2024

Conversation

dfbakin
Copy link
Member

@dfbakin dfbakin commented Mar 23, 2024

Test

Main changes

  • Located lock-free queue to a separate package
  • Added tests for correctness and stability -- passed without fixes
  • Tried to link shared library out of template header file -- in progress

@dfbakin dfbakin added the refactoring Codestyle / minor changes label Mar 23, 2024
@dfbakin dfbakin self-assigned this Mar 23, 2024
@dfbakin dfbakin marked this pull request as ready for review March 23, 2024 17:53
@dfbakin dfbakin requested a review from dasimagin March 23, 2024 17:53
dasimagin
dasimagin approved these changes Apr 14, 2024
View reviewed changes
packages/lock_free_queue/CMakeLists.txt Outdated Show resolved Hide resolved
packages/lock_free_queue/CMakeLists.txt Outdated Show resolved Hide resolved
packages/lock_free_queue/CMakeLists.txt Outdated Show resolved Hide resolved
packages/camera/CMakeLists.txt Outdated Show resolved Hide resolved
@dfbakin
fix: requested changes implemented
95834c0 
fix: added header-only include location correctly
@dfbakin dfbakin merged commit bdc53cf into master Apr 14, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dasimagin dasimagin dasimagin approved these changes

Assignees

@dfbakin dfbakin

Labels
refactoring Codestyle / minor changes
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dfbakin @dasimagin