[FFmpeg-devel] [PATCH 5/6] mxfdec: Only sort IndexTableSegments once

Michael Niedermayer michaelni at gmx.at
Mon Dec 12 18:31:46 CET 2011


On Mon, Dec 12, 2011 at 10:20:13AM +0100, Tomas Härdin wrote:
> The index tables are the same/apply to all streams, so we only need to
> sort them once.
> Or put more simply: The ordering doesn't depend on which stream we're
> computing the index for.
>
> /Tomas

>  mxfdec.c |   58 ++++++++++++++++++++++++++++------------------------------
>  1 file changed, 28 insertions(+), 30 deletions(-)
> 5e74faae6eb375fcf31842f9c74a5bd83ea509c2  0005-mxfdec-Only-sort-IndexTableSegments-once.patch
> From cef116128bd2d7db50b2a78f61214b8a1864936a Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Tomas=20H=C3=A4rdin?= <tomas.hardin at codemill.se>
> Date: Fri, 9 Dec 2011 17:17:33 +0100
> Subject: [PATCH 5/6] mxfdec: Only sort IndexTableSegments once

Applied, thanks

[...]

--
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111212/3a3ee2d6/attachment.asc>


More information about the ffmpeg-devel mailing list