Re: [FFmpeg-devel] [FFmpeg-cvslog] ffmpeg: add per-stream input option drop_changed

[James Almer <jamrial@gmail.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1452 bytes --]

On 3/24/2025 9:31 AM, Martin Storsjö wrote:
> On Mon, 24 Mar 2025, Gyan Doshi wrote:
> 
>> ffmpeg | branch: master | Gyan Doshi <ffmpeg@gyani.pro> | Sat Mar 15 
>> 16:51:05 2025 +0530| [cbbc927a67f142a02bfc5d047c03587599ee5b55] | 
>> committer: Gyan Doshi
>>
>> ffmpeg: add per-stream input option drop_changed
>>
>> This is a replacement in ffmpeg for the deprecated avcodec flag 
>> AV_CODEC_FLAG_DROPCHANGED.
>>
>> This option is meant to be used when the filtergraph should not be
>> reinited upon input parameter changes as that leads to loss of state
>> in the filtergraph potentially leading to broken or aborted output,
>> e.g. inserting of silence with first_pts specified in aresample.
>>
>> Generally useful to avoid corrupted yet decodable packets in live
>> streaming inputs.
>>
>> This option when enabled takes precedence over reinit_filters
>>
>>> http://git.videolan.org/gitweb.cgi/ffmpeg.git/? 
>>> a=commit;h=cbbc927a67f142a02bfc5d047c03587599ee5b55
>> ---
> 
> This change is causing failed fate tests in many different 
> configurations; I'm seeing null pointer dereferences and similar, on 
> both msvc/arm, msvc/x86_32, macos/aarch64. Please investigate.

I pushed a quick fix, but nonetheless, the log message in question is 
being printed for one stream (the last one handled by the scheduler) 
rather than every stream where the option was the for, so it needs to be 
properly done still.


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".