Le 26 novembre 2024 05:05:53 GMT+02:00, Sean McGovern <gsean...@gmail.com> a écrit : >Hi Rémi, > >On Mon, Nov 25, 2024, 13:02 Rémi Denis-Courmont <r...@remlab.net> wrote: > >> Le sunnuntaina 24. marraskuuta 2024, 21.01.48 EET Sean McGovern a écrit : >> > The FATE test 'checkasm-sw_yuv2rgb' currently fails on this platform, >> > in both little- and big-endian configurations. >> > >> > Disable it by default. >> > Add '-DSWS_USE_ALTIVEC_YUV2RGB' to CPPFLAGS to re-enable it. >> > --- >> > v2: rebased over top of the recent swscale changes from Nik Haas >> > --- >> > libswscale/ppc/yuv2rgb_altivec.c | 2 ++ >> > 1 file changed, 2 insertions(+) >> > >> > diff --git a/libswscale/ppc/yuv2rgb_altivec.c >> > b/libswscale/ppc/yuv2rgb_altivec.c index 57574c4ab3..f44733a880 100644 >> > --- a/libswscale/ppc/yuv2rgb_altivec.c >> > +++ b/libswscale/ppc/yuv2rgb_altivec.c >> > @@ -558,6 +558,7 @@ av_cold SwsFunc ff_yuv2rgb_init_ppc(SwsInternal *c) >> > if ((c->opts.src_h & 0x1) != 0) >> > return NULL; >> > >> > +#ifdef SWS_USE_ALTIVEC_YUV2RGB >> >> TBH, I find this somwhat misleading as opposed to be a plain 'if 0' with a >> rationale comment. >> > >I can do that but I feel like I would just be re-stating the commit message.
I don't know about you, but I don't typically see commit messages when looking at the source code. So unless I specifically want to look up commit messages and run git-blame to find them, they're invisible. So I don't get your point. _______________________________________________ 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".
