[FFmpeg-devel] [PATCH] yuv2rgb_neon: fix return value

Martin Storsjö martin at martin.st
Tue Jul 7 22:58:07 EEST 2020


On Tue, 7 Jul 2020, Lynne wrote:

> We return 0 for this particular architecture but should instead be
> returning the number of lines.
> Fixes users who check the return value matches what they expect.

The change looks good in itself, but it also looks like we have the same 
issue in the arm version of the same assembly, right?

I presume we don't have a preexisting checkasm test for this function, 
where we could add a check for the return value (which would expose all 
other instances of the same issue)?

// Martin



More information about the ffmpeg-devel mailing list