[FFmpeg-devel] [PATCH 2/5] truehd: break out part of rematrix_channels into platform-specific callback.

Michael Niedermayer michaelni at gmx.at
Wed Mar 26 20:43:16 CET 2014


On Thu, Mar 20, 2014 at 06:59:15PM +0000, Ben Avison wrote:
> Verified with profiling that this doesn't have a measurable effect upon
> overall performance.
> ---
>  libavcodec/mlpdec.c |   37 ++++++++++++-------------------------
>  libavcodec/mlpdsp.c |   33 +++++++++++++++++++++++++++++++++
>  libavcodec/mlpdsp.h |   23 +++++++++++++++++++++++
>  3 files changed, 68 insertions(+), 25 deletions(-)

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/20140326/ad7c00f0/attachment.asc>


More information about the ffmpeg-devel mailing list