[FFmpeg-devel] [PATCH] Bink Video decoder for FFmpeg 0.6
Kostya
kostya.shishkov
Wed Feb 17 18:56:54 CET 2010
On Wed, Feb 17, 2010 at 06:07:37AM -0800, Jason Garrett-Glaser wrote:
> >> + ? ? ? ?if (!plane || !c->swap_planes)
> >> + ? ? ? ? ? ?plane_idx = plane;
> >> + ? ? ? ?else
> >> + ? ? ? ? ? ?plane_idx = plane ^ 3;
> >>
> >> I'd ternary this.
> >
> > Why?
>
> Cleaner, shorter.
ok
> > Mainly because quant table index must be read after all DCT coefficients
> > are decoded (sounds like bad design to me).
>
> Sounds like you need a dequant DSP function then. This would benefit
> greatly from asm.
>
> You could also track the last nnz and only dequant up to it.
I've managed to merge dequant with coefficient decoding instead.
> Dark Shikari
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bink-v2.patch
Type: text/x-diff
Size: 79777 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100217/f019893b/attachment.patch>
More information about the ffmpeg-devel
mailing list