[FFmpeg-devel] [PATCH]Audio support for mplayer -demuxer lavf and vivo

[Paul B Mahol]

On 11/29/12, Peter Ross <pross at xvid.org> wrote:
> On Thu, Nov 29, 2012 at 10:23:20AM +0000, Paul B Mahol wrote:
>> On 11/29/12, Peter Ross <pross at xvid.org> wrote:
>> > On Thu, Nov 29, 2012 at 09:52:34AM +0000, Paul B Mahol wrote:
>> >> On 11/29/12, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
>> >> > Paul B Mahol <onemda <at> gmail.com> writes:
>> >> >
>> >> >> >> > +    AV_CODEC_ID_VIVOAUDIO   = MKBETAG('V','I','V',0xA),
>> >> >> >>
>> >> >> >> This one appears have real name and twocc:
>> >> >> >>
>> >> >> >> 0x112 Vivo Siren
>
> Hi, is there a riff/wave sample of this somewhere?
>
>> >> >> >
>> >> >> > So do you prefer:
>> >> >> > AV_CODEC_ID_VIVO_SIREN = MKBETAG('V','I','V',0xA),
>> >> >> > ?
>> >> >> >
>> >> >> > (How is the twocc related to the codec_id entry?)
>> >> >>
>> >> >> It is not. I was just mentioned it because of theoretical
>> >> >> posibility
>> >> >> it can be found in avi/wav.
>> >> >
>> >> > Thank you for explaining.
>> >> >
>> >> > Is AV_CODEC_ID_VIVO_SIREN ok?
>
> AV_CODEC_ID_SIREN

Agree.
>
>> >>
>> >> I really see no reason why you are pushing this before actual decoder
>> >> is written. MPlayer can still use internal vivo demuxer and not lavf
>> >> one.
>> >
>> > Is it Siren, or a special VIVO brand of Siren?
>>
>> Isn't siren7 created much later. After all you could try it.
>
> Libsiren7 supports 4 variants of siren:
>
> VIVO uses variant 0: sample rate is not stored in the header.
>
> MSN (RIFF 0x028E) uses variant 1: sample rate is stored in header, and does
> not change.
>    MSN files come with extradata {0x40, 0x01}, so this could be used to
> differeniate
>    them from VIVO.
>
> The Libsiren7 decoder default to variant 1. It correctly decodes the ffmpeg
> VIVO samples
> if one changes 'GetSirenCodecInfo(1, ...) to 'GetSirenCodecInfo(0, ...)
> (libsiren7/common.c).

Nice! You gonna port libsiren7?