diff mbox series

[FFmpeg-devel,07/20] avformat/sccdec: Remove redundant check

Message ID AM7PR03MB6660ADBEDAC93BFFEB0B13D28FAB9@AM7PR03MB6660.eurprd03.prod.outlook.com
State Accepted
Commit e1d2a208a9be655062b57a19337447631f55e203
Headers
Series [FFmpeg-devel,01/20] libpostproc/postprocess_template: Don't reimplement FFSWAP |

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Andreas Rheinhardt Oct. 1, 2021, 9:08 p.m. UTC 
The av_sscanf() will filter lines like "Scenarist_SCC V1.0" out.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavformat/sccdec.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Paul B Mahol Oct. 2, 2021, 2:58 p.m. UTC | #1 
lgtm i guess
diff mbox series

Patch

diff --git a/libavformat/sccdec.c b/libavformat/sccdec.c
index 1861a99aad..efd6024001 100644
--- a/libavformat/sccdec.c
+++ b/libavformat/sccdec.c
@@ -88,8 +88,6 @@  static int scc_read_header(AVFormatContext *s)
                 break;
             continue;
         }
-                if (!strncmp(line, "Scenarist_SCC V1.0", 18))
-                    continue;
         if (av_sscanf(line, "%d:%d:%d%*[:;]%d", &hh, &mm, &ss, &fs) != 4)
             continue;