[FFmpeg-devel] [PATCH]lavf/amr: Add amrnb and amrwb demuxers

[Paul B Mahol]

On 11/5/17, Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:
> 2017-10-09 0:24 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>> 2017-10-07 16:51 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>>> 2017-10-01 18:23 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>>>> 2017-09-27 18:08 GMT+02:00 Carl Eugen Hoyos <ceffmpeg at gmail.com>:
>>>>
>>>>> The existing amr demuxer does not allow reading streams,
>>>>> it requires the 3GPP-conforming file header.
>>>>> Attached patch allows reading amrnb and amrwb from (live)
>>>>> streams, fixes ticket #6678.
>>>>
>>>> New patch with auto-detection attached, passes probecheck.
>>>
>>> Simplified patch attached that does not duplicate two small arrays
>>> in the object file that are already duplicated in the source code.
>>> The uninitialized variable is also fixed.
>>
>> The last version triggered for repeated bytes (like adp),
>> new version attached.
>
> Patch applied.

Please remove codec_tag line you set in those demuxers.
Codec tag makes sense to set only and only if container have such tag.
Raw containers nowhere stores tags so no reason to set it to .mov values
either.