[FFmpeg-devel] [PATCH] avfilter/vf_libvmaf: update filter for libvmaf v2.0.0

Kyle Swanson k at ylo.ph
Sun Jan 23 23:22:24 EET 2022


Hi,

On Thu, Jan 20, 2022 at 1:06 PM Kyle Swanson <k at ylo.ph> wrote:
>
> Hi,
>
> On Wed, Jan 19, 2022 at 10:23 AM Kyle Swanson <k at ylo.ph> wrote:
> >
> > Hi,
> >
> > On Thu, Jan 13, 2022 at 6:38 PM Kyle Swanson <k at ylo.ph> wrote:
> > >
> > > Hi,
> > >
> > > On Mon, Jan 10, 2022 at 10:22 AM Andreas Rheinhardt
> > > <andreas.rheinhardt at outlook.com> wrote:
> > > > 1. FFmpeg uses the ancient C90 rule that only allows variable
> > > > declarations at the beginning of a block (before all statements). You
> > > > should actually have received a compiler warning because of the above
> > > > code (unless you use Clang, which claims to support this type of
> > > > warning, yet doesn't).
> > > > 2. You jump over the initializion of str_copy if the dict allocation
> > > > fails and consequently you free an uninitialized string.
> > >
> > > New patch attached. Fixes all of the GCC -Wdeclaration-after-statement
> > > warnings. You're right, I didn't notice because I was building with
> > > Clang. Andreas, I will just wait for your LGTM before merging.
> > >
> > > Thanks,
> > > Kyle
> >
> > Updated patch attached. Thank you to mkver for the mini review on IRC.
> > The `delimited_dict_parse` multi-delimiter corner case should be
> > sorted now.
> >
> > Thanks,
> > Kyle
>
> Will push this soon.
>
> Thanks,
> Kyle

Pushed.

Thanks,
Kyle


More information about the ffmpeg-devel mailing list