On Wed, Aug 28, 2024 at 02:14:33PM +0200, Ramiro Polla wrote:
> 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?

If you tested it and it works it has to be correct.
This would result in significant errors if it was wrong

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus

Attachment: signature.asc
Description: PGP signature

_______________________________________________
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