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