Re: [FFmpeg-devel] [PATCH v2] lavf/mpegts: add supplementary audio descriptor

2018-02-20 Thread Stefan Pöschel
Am 20.02.2018 um 00:13 schrieb Marton Balint: >>> +    if (ext_desc_tag == 0x06) { /* supplementary audio descriptor */ >>> +    char flags; > > Change this to int, I see no reason why this is a char. OK, I will fix that with v3. >> LGTM. Will commit in a couple days along with my

Re: [FFmpeg-devel] [PATCH v2] lavf/mpegts: add supplementary audio descriptor

2018-02-19 Thread Marton Balint
On Mon, 19 Feb 2018, Aman Gupta wrote: On Fri, Feb 16, 2018 at 11:22 AM, Stefan Pöschel wrote: The supplementary audio descriptor is defined in ETSI EN 300 468 and provides more details regarding accessibility audio tracks, especially the normative annex J contains a

Re: [FFmpeg-devel] [PATCH v2] lavf/mpegts: add supplementary audio descriptor

2018-02-19 Thread Aman Gupta
On Fri, Feb 16, 2018 at 11:22 AM, Stefan Pöschel wrote: > The supplementary audio descriptor is defined in ETSI EN 300 468 and > provides more details regarding accessibility audio tracks, especially > the normative annex J contains a detailed description of its use. > > Its