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

[core] Added atomic support and marked atomic key fields detected by thread sanitizer #1863

Merged
merged 11 commits into from
Jun 1, 2021
Merged

[core] Added atomic support and marked atomic key fields detected by thread sanitizer #1863

merged 11 commits into from
Jun 1, 2021

Conversation

ethouris
Copy link
Collaborator

  1. Added atomic support using compiler intrinsics-based external library (in C++11 mode it uses standard atomic)
  2. Added support for atomic chrono types (time_point and duration). This was needed as well for C++11 chrono types.
  3. Marked required fields atomic.

@ethouris ethouris mentioned this pull request Mar 12, 2021
Open
23 tasks
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Mar 12, 2021

This pull request introduces 1 alert when merging 9039b3c into 2ef0c8d - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Mar 22, 2021

This pull request introduces 1 alert when merging d3147d7 into 449d917 - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@ethouris ethouris mentioned this pull request Apr 15, 2021
Closed
@maxsharabayko maxsharabayko mentioned this pull request Apr 19, 2021
Closed
@maxsharabayko maxsharabayko added [core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests labels Apr 28, 2021
@maxsharabayko maxsharabayko added this to the v1.4.4 milestone Apr 28, 2021
@lgtm-com
Copy link
Contributor

lgtm-com bot commented May 28, 2021

This pull request introduces 1 alert when merging 4c8f794 into e2a00aa - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@lgtm-com
Copy link
Contributor

lgtm-com bot commented May 28, 2021

This pull request introduces 1 alert when merging 880b07f into e2a00aa - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Jun 1, 2021

This pull request introduces 1 alert when merging 80cef25 into 1751bab - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@ethouris ethouris marked this pull request as ready for review June 1, 2021 11:48
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Jun 1, 2021

This pull request introduces 1 alert when merging 349bc49 into e37f4ab - view on LGTM.com

new alerts:

  • 1 for Overloaded assignment does not return 'this'

@maxsharabayko maxsharabayko merged commit 0f0caf9 into Haivision:master Jun 1, 2021
@maxsharabayko maxsharabayko mentioned this pull request Jul 17, 2021
Closed
@maxsharabayko maxsharabayko mentioned this pull request Dec 9, 2021
Merged
@mbakholdina mbakholdina mentioned this pull request Dec 10, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
[core] Area: Changes in SRT library core Type: Enhancement Indicates new feature requests
Projects
None yet
Milestone
v1.4.4
Development

Successfully merging this pull request may close these issues.
2 participants
@ethouris @maxsharabayko