[FFmpeg-devel] [PATCH] h264: hl_decode_mb_internal optimisation

Michael Niedermayer michaelni
Wed Jul 30 15:09:38 CEST 2008

On Mon, Jul 28, 2008 at 05:51:47PM +1200, Paul Kendall wrote:
> In the hl_decode_mb_internal function, the access pattern for h->mb is 
> seqential, so we can simplify the code to get the value into a local and use 
> *mb++ pattern.
> Also, we can simplify the copy destination logic by calculating values at 
> various parts of the loop rather then for each store action.

PCM macroblocks should occur VERY rarely thus i dont think this will have
any effect on speed not even if the code would be 10 times faster.
If not speed then its just readability and i think old and new variants
are rather similar readability wise.

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is dangerous to be right in matters on which the established authorities
are wrong. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080730/0401e9d5/attachment.pgp>

More information about the ffmpeg-devel mailing list