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

Paul B Mahol onemda at gmail.com
Mon Nov 6 12:50:08 EET 2017


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.


More information about the ffmpeg-devel mailing list