On Thu, Aug 22, 2024 at 1:24 AM Ramiro Polla <ramiro.po...@gmail.com> wrote:
> The x86 optimized dct_quantize only calculates the last nonzero
> coefficient correctly if the zigzag scan order is used. For the
> alternate scan order, this value is incorrect.
>
> To work around this, the dct_unquantize functions process the entire
> block if the alternate scan order is used.
>
> But a second workaround (bb198e198ab) was added that recalculates the
> last nonzero coefficient after dct_quantize is called if the alternate
> scan order is used.
>
> This commit removes the first workaround, which became redundant.
> ---
>  libavcodec/mpegvideo.c     | 9 +++------
>  libavcodec/x86/mpegvideo.c | 6 ++----
>  2 files changed, 5 insertions(+), 10 deletions(-)

Michael, could you please check if my analysis and the changes are correct?

Thanks,
Ramiro
_______________________________________________
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".

Reply via email to