On Mon, 07 Oct 2024 00:15:32 +0200 Michael Niedermayer <[email protected]> wrote: > On Sat, Oct 05, 2024 at 09:23:55PM +0200, Niklas Haas wrote: > > From: Niklas Haas <[email protected]> > > > > 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 <[email protected]> > > --- > > 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? I am fine dropping this commit for now. > > thx > > [...] > > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > Many things microsoft did are stupid, but not doing something just because > microsoft did it is even more stupid. If everything ms did were stupid they > would be bankrupt already. > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > [email protected] with subject "unsubscribe". _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
