awslabs / amazon-kinesis-video-streams-pic

Apache License 2.0
49 stars 47 forks source link

fix various tsan issues related to viewItemRemove and TO_STREAM_HANDL… #241

Closed hassanctech closed 7 months ago

hassanctech commented 7 months ago

Issue #, if available:

Description of changes:

Fix various tsan issues related to viewItemRemove and TO_STREAM_HANDLE which access the stream list an an index

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

codecov-commenter commented 7 months ago

Codecov Report

Attention: 28 lines in your changes are missing coverage. Please review.

Comparison is base (07dea9e) 80.36% compared to head (bd12675) 80.27%.

:exclamation: Current head bd12675 differs from pull request most recent head 0ccd2e6. Consider uploading reports for the commit 0ccd2e6 to get more accurate results

Files Patch % Lines
src/client/src/Stream.c 70.17% 17 Missing :warning:
src/client/src/Client.c 63.33% 11 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #241 +/- ## =========================================== - Coverage 80.36% 80.27% -0.09% =========================================== Files 53 53 Lines 10804 10877 +73 =========================================== + Hits 8683 8732 +49 - Misses 2121 2145 +24 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.