[FFmpeg-devel] [PATCH] avformat/oggparseogm: sync avctx w/ codecpar

James Almer jamrial at gmail.com
Tue Jun 25 20:11:42 EEST 2019


On 6/25/2019 1:44 PM, Chris Cunningham wrote:
> Friendly ping.
> 
> On Thu, Jun 20, 2019 at 11:17 AM Chris Cunningham <chcunningham at chromium.org>
> wrote:
> 
>> On Thu, Feb 28, 2019 at 9:13 AM James Almer <jamrial at gmail.com> wrote:
>>
>>> On 2/26/2019 10:18 PM, Chris Cunningham wrote:
>>>> On Thu, Feb 21, 2019 at 4:46 PM Chris Cunningham
>>>> <chcunningham at chromium.org <mailto:chcunningham at chromium.org>> wrote:
>>>>
>>>>     I'm fine to do either. James, do you still prefer to skip the later
>>>>     headers if this breaks some old ogm files?
>>>>
>>>>
>>>> James, friendly ping
>>>
>>> Yes, i'd prefer if we skip any superfluous parameter header that shows
>>> up before the first data packet.
>>> The return value of ff_codec_get_id() should also be checked to not
>>> propagate AV_CODEC_ID_NONE, which was the source of this issue.
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel at ffmpeg.org
>>> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>>
>> Renewing request to apply for this patch.
>>
>> The alternate route of identifying/skipping the bad packets was more
>> difficult than expected (https://patchwork.ffmpeg.org/patch/13593/).
>> James signed on this approach instead.
>>
>> Chris

Applied.


More information about the ffmpeg-devel mailing list