On date Saturday 2016-04-02 15:56:55 +0200, Michael Niedermayer encoded:
> On Sat, Apr 02, 2016 at 12:46:57PM +0200, Stefano Sabatini wrote:
> > This allows to recognize ID3 packets from the stream type.
> > ---
> >  libavformat/mpegts.c | 1 +
> >  1 file changed, 1 insertion(+)
> 
> breaks tickets/2579/old_klv_data_stream.mpg
> 
> "breaks" in the sense of detecting the klv stuff as id3

Previous patch was indeed broken. New patch attached.
-- 
FFmpeg = Fiendish & Faithful Magical Problematic Educated Gymnast
>From a756486700d520d7c1e50128c731a59177cc3545 Mon Sep 17 00:00:00 2001
From: Stefano Sabatini <stefa...@gmail.com>
Date: Thu, 7 Apr 2016 20:40:52 +0200
Subject: [PATCH] lavf/mpegts: add ID3 entry to the REGD_types array

This allows to recognize ID3 packets from their corresponding descriptor
tag.
---
 libavformat/mpegts.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
index e44da1f..1258913 100644
--- a/libavformat/mpegts.c
+++ b/libavformat/mpegts.c
@@ -739,6 +739,7 @@ static const StreamType REGD_types[] = {
     { MKTAG('D', 'T', 'S', '3'), AVMEDIA_TYPE_AUDIO, AV_CODEC_ID_DTS   },
     { MKTAG('H', 'E', 'V', 'C'), AVMEDIA_TYPE_VIDEO, AV_CODEC_ID_HEVC  },
     { MKTAG('K', 'L', 'V', 'A'), AVMEDIA_TYPE_DATA,  AV_CODEC_ID_SMPTE_KLV },
+    { MKTAG('I', 'D', '3', ' '), AVMEDIA_TYPE_DATA,  AV_CODEC_ID_TIMED_ID3 },
     { MKTAG('V', 'C', '-', '1'), AVMEDIA_TYPE_VIDEO, AV_CODEC_ID_VC1   },
     { MKTAG('O', 'p', 'u', 's'), AVMEDIA_TYPE_AUDIO, AV_CODEC_ID_OPUS  },
     { 0 },
-- 
1.9.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to