mirror of
https://github.com/CCExtractor/ccextractor.git
synced 2026-04-30 02:25:54 +00:00
[PR #1269] [MERGED] [FIX] -Wunused-result warnings #2056
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/CCExtractor/ccextractor/pull/1269
Author: @NilsIrl
Created: 5/20/2020
Status: ✅ Merged
Merged: 6/28/2020
Merged by: @cfsmp3
Base:
master← Head:result_warnings📝 Commits (4)
3b77242Fix -Wunused-result warnings8ad96c0Wrap checked writes into a function3977715In write_wrapped, continue writing in case of partial write90929dfFix build on MSVC📊 Changes
14 files changed (+166 additions, -149 deletions)
View changed files
📝
src/lib_ccx/ccx_decoders_708_output.c(+23 -22)📝
src/lib_ccx/ccx_encoders_g608.c(+7 -7)📝
src/lib_ccx/ccx_encoders_mcc.c(+2 -2)📝
src/lib_ccx/ccx_encoders_sami.c(+14 -17)📝
src/lib_ccx/ccx_encoders_scc.c(+9 -8)📝
src/lib_ccx/ccx_encoders_smptett.c(+13 -13)📝
src/lib_ccx/ccx_encoders_srt.c(+15 -15)📝
src/lib_ccx/ccx_encoders_ssa.c(+12 -12)📝
src/lib_ccx/ccx_encoders_transcript.c(+1 -1)📝
src/lib_ccx/ccx_encoders_webvtt.c(+19 -19)📝
src/lib_ccx/file_functions.c(+2 -1)📝
src/lib_ccx/matroska.c(+34 -32)📝
src/lib_ccx/utility.c(+13 -0)📝
src/lib_ccx/utility.h(+2 -0)📄 Description
In raising this pull request, I confirm the following (please check boxes):
My familiarity with the project is as follows (check one):
Remove -Wunused-result warnings. The main reason to do this is that contributors aren't flooded with warnings which cause them to ignore them.
output diff:
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.