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 _______________________________________________ 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".
