[FFmpeg-devel] [PATCH 1/2] avutil: Add Simple loop detector

Lynne dev at lynne.ee
Thu Aug 8 15:29:42 EEST 2019


Aug 8, 2019, 11:36 AM by george at nsup.org:

> Lynne (12019-08-08):
>
>> That's very niche and I don't think it belongs to lavu.
>>
>
> That's a single header with no compiled code.
>
>> Having codec specific code is better.
>>
>
> Why? Why risk bug duplication and maintenance burden?
>

Its niche, we have very few places where there's a chance a loop will become infinite. Even then, they're likely all in lavc, and you can just put a non-public header there if there's more than a few. And right now, there's only one.
Plus, its not really common av code, and don't let its name mislead you, its more of a hack.


More information about the ffmpeg-devel mailing list