[FFmpeg-devel] [RFC] Reviewing merges

[Marton Balint]

On Mon, 24 Apr 2017, Michael Niedermayer wrote:
> On Mon, Apr 24, 2017 at 11:23:16AM -0300, James Almer wrote:

>> We have recently been able to go through six hundred or so commits in a
>> month or two this way after being stuck for the longest time by a few of
>> those big API changes. If we start requiring every commit to go through
>> a review process on the ML then we will never catch up with the backlog.
>> In short, things as they are right now are smooth. Changing it will only
>> make this slower.
>
> Maybe, but is merging more faster also better for FFmpeg ?
> I did not analyze the bugs on our bug tracker but subjectivly the
> number of regressions seems much larger than a year or 2 ago.
> and i just yesterday found 2 issues in a merge (which you fixed)
>

Yeah, I also have two I recently came across, both caused by the delayed 
filter initialization patch:

https://trac.ffmpeg.org/ticket/6323
https://trac.ffmpeg.org/ticket/6318

Maybe someone more familiar with ffmpeg.c code can take a look?

Thanks,
Marton