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

Fix the error of not accumulating nwrite in srs_write_large_iovs #1651

Merged
merged 2 commits into from
Mar 21, 2020
Merged

Fix the error of not accumulating nwrite in srs_write_large_iovs #1651

merged 2 commits into from
Mar 21, 2020

Conversation

lam2003
Copy link
Contributor

@lam2003 lam2003 commented Mar 19, 2020


TRANS_BY_GPT3

@codecov-io
Copy link

codecov-io commented Mar 19, 2020

Codecov Report

Merging #1651 into 3.0release will decrease coverage by 0.03%.
The diff coverage is 75%.

Impacted file tree graph

@@              Coverage Diff               @@
##           3.0release    #1651      +/-   ##
==============================================
- Coverage       52.89%   52.86%   -0.04%     
==============================================
  Files              82       82              
  Lines           26619    26622       +3     
==============================================
- Hits            14080    14073       -7     
- Misses          12539    12549      +10

| Impacted Files | Coverage Δ | |'

Translated to English while maintaining the markdown structure:

'| Impacted Files | Coverage Δ | |
|---|---|---|
| trunk/src/protocol/srs_protocol_utility.cpp | 96.04% <75%> (-0.51%) | ⬇️ |
| trunk/src/core/srs_core_autofree.hpp | 0% <0%> (-100%) | ⬇️ |


Continue to review full report at Codecov.

Legend - Click here to learn more
Translated to English while maintaining the markdown structure:

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

Powered by Codecov. Last update aead1a5...bbfb03f. Read the comment docs.

TRANS_BY_GPT3

@winlinvip winlinvip merged commit 355f351 into ossrs:3.0release Mar 21, 2020
@winlinvip
Copy link
Member

Add utest to cover code in f89b4b3

@winlinvip winlinvip changed the title 修复srs_write_large_iovs中nwrite未累加的错误 Fix the error of not accumulating nwrite in srs_write_large_iovs Jul 29, 2023
@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.

3 participants