On Mon, 07 Oct 2024 20:03:06 +0200 Michael Niedermayer <mich...@niedermayer.cc> 
wrote:
> On Mon, Oct 07, 2024 at 06:47:24PM +0200, Niklas Haas wrote:
> > On Mon, 07 Oct 2024 00:15:32 +0200 Michael Niedermayer 
> > <mich...@niedermayer.cc> wrote:
> > > On Sat, Oct 05, 2024 at 09:23:55PM +0200, Niklas Haas wrote:
> > > > From: Niklas Haas <g...@haasn.dev>
> > > >
> > > > This is done for consistency with the other public enums which will be
> > > > added in the upcoming swscale API refactor. I went through the code and 
> > > > checked
> > > > carefully that the value of `dither` is never implicitly compared 
> > > > against
> > > > zero, so this change should not break anything.
> > > >
> > > > Sponsored-by: Sovereign Tech Fund
> > > > Signed-off-by: Niklas Haas <g...@haasn.dev>
> > > > ---
> > > >  libswscale/swscale_internal.h | 4 ++--
> > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > this breaks:
> > >
> > > ./ffmpeg -i lena.pnm -vf 
> > > scale=iw:ih:flags=0x000002:sws_dither=0,format=bgr4_byte -bitexact 
> > > whatever.bmp
> >
> > This is expected, of course. Are you saying that the change in consistency 
> > is
> > not worth the risk of breaking users who directly specify numeric IDs 
> > instead
> > of using the symbolic names?
>
> iam not really sure what to do but
> 0 has a feeling of "none" to me

Well, I also wanted to add a "none" option to the enum, so maybe I can make
that be the value of 0. I just think that it's weird for the default value of
an enum option to be something other than 0, and the default should probably
be "auto". (Otherwise, what's the point of the "auto" option?)

>
> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> It is what and why we do it that matters, not just one of them.
> _______________________________________________
> 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