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.

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

SRT: Use thread-safe log for multiple-threading SRT module. #2474

Merged
merged 15 commits into from
Jul 21, 2021
Merged

SRT: Use thread-safe log for multiple-threading SRT module. #2474

merged 15 commits into from
Jul 21, 2021

Conversation

runner365
Copy link
Contributor

add srt new log system:
srt log message will be sent to srs st thread to print.

@runner365 runner365 requested review from xiaozhihong, winlinvip and a team July 14, 2021 15:39
@codecov-commenter
Copy link

codecov-commenter commented Jul 14, 2021

Codecov Report

Merging #2474 (d888903) into develop (12ba584) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2474   +/-   ##
========================================
  Coverage    42.66%   42.66%           
========================================
  Files          102      102           
  Lines        36039    36038    -1     
========================================
+ Hits         15376    15377    +1     
+ Misses       20663    20661    -2     

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/kernel/srs_kernel_rtc_rtcp.cpp | 1.98% <ø> (+<0.01%) | ⬆️ |

Translated to English while maintaining the markdown structure:

| trunk/src/kernel/srs_kernel_rtc_rtcp.cpp | 1.98% <ø> (+<0.01%) | ⬆️ |
| trunk/src/protocol/srs_service_utility.cpp | 72.97% <0.00%> (+0.54%) | ⬆️ |


Continue to review full report at Codecov.

Legend - Click here to learn more
| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Translated to English while maintaining the markdown structure:

| Δ = absolute <relative> (impact), ø = not affected, ? = missing data |

Powered by Codecov. Last update 12ba584...d888903. Read the comment docs.

TRANS_BY_GPT3

@winlinvip winlinvip changed the title Srt new log SRT: Use thread-safe log for multiple-threading SRT module. Jul 21, 2021
@winlinvip winlinvip merged commit bdd02e1 into ossrs:develop Jul 21, 2021
winlinvip pushed a commit that referenced this pull request Jul 21, 2021
* solve srt push bugs

* solve h264 mutiple nalus in srt when obs is configured in zerolatency

* optimize error code

* optimize error code

* optimize error code

* add commemnt:we only skip pps/sps frame and send left nalus in srt

* add commemnt:we only skip pps/sps frame and send left nalus in srt

* optimize srt log system

* update conf

* update srt hpp

Co-authored-by: shiwei <shiwei05@kuaishou.com>
winlinvip pushed a commit that referenced this pull request Jul 21, 2021
…4.0.143

* solve srt push bugs

* solve h264 mutiple nalus in srt when obs is configured in zerolatency

* optimize error code

* optimize error code

* optimize error code

* add commemnt:we only skip pps/sps frame and send left nalus in srt

* add commemnt:we only skip pps/sps frame and send left nalus in srt

* optimize srt log system

* update conf

* update srt hpp

Co-authored-by: shiwei <shiwei05@kuaishou.com>
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants