On Fri, 15 Nov 2024 00:49:06 +0100 Michael Niedermayer <mich...@niedermayer.cc> 
wrote:
> On Fri, Nov 15, 2024 at 12:11:34AM +0100, Niklas Haas wrote:
> > On Fri, 15 Nov 2024 00:00:10 +0100 Michael Niedermayer 
> > <mich...@niedermayer.cc> wrote:
> > > On Tue, Nov 12, 2024 at 10:50:46AM +0100, Niklas Haas wrote:
> > > > From: Niklas Haas <g...@haasn.dev>
> > > >
> > > > Most logic from this filter has been co-opted into swscale itself,
> > > > allowing the resulting filter to be substantially simpler as it no
> > > > longer has to worry about context initialization, interlacing, etc.
> > > >
> > > > Sponsored-by: Sovereign Tech Fund
> > > > Signed-off-by: Niklas Haas <g...@haasn.dev>
> > > > ---
> > > >  libavfilter/vf_scale.c | 354 +++++++++--------------------------------
> > > >  1 file changed, 72 insertions(+), 282 deletions(-)
> > >
> > > ./ffmpeg -i foreman_cif.y4m  -vf scale=out_v_chr_pos=0:out_h_chr_pos=0 -f 
> > > null -
> > >
> > > [fc#-1 @ 0x55eec4ea3300] Error applying option 'out_v_chr_pos' to filter 
> > > 'scale': Option not found
> > > Error opening output file -.
> > > Error opening output files: Option not found
> >
> > I mean, this change is basically intentional. But I suppose I should add
> > backwards compatibility code to at least round it to the nearest similar 
> > value.
>
> yes, it should do something better than just failing with a message that 
> confuses the user
>
> also [fc#-1 @ 0x55b30de5f000] Error applying option 'in_v_chr_pos' to filter 
> 'scale': Option not found
> and others

Decided to keep it in place as a legacy option for now. Removing it seems like
a better fit for a later milestone, alongside other legacy stuff.

>
> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> I know you won't believe me, but the highest form of Human Excellence is
> to question oneself and others. -- Socrates
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

Reply via email to