[FFmpeg-devel] [PATCH] lls: move the Cholesky decomposition in a separate function

Michael Niedermayer michaelni
Sat Oct 31 23:07:49 CET 2009


On Sat, Oct 31, 2009 at 02:24:11PM +0100, Nicolas George wrote:
> Hi.
> 
> The attached patch moves the Cholesky decomposition of a matrix in a
> separate function and documents it.

The question i have is, why?

Also you completely rewrite the code during the move, making it much more
complex, harder to read (for me) and bigger (in terms of lines of code)
ahh, and your patch adds non ascii chars, i think there was some kind of
consensus amongth the ffmpeg devels that non ascii should be limited to
author names or some translated messages

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

I wish the Xiph folks would stop pretending they've got something they
do not.  Somehow I fear this will remain a wish. -- M?ns Rullg?rd
-------------- 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/20091031/6c507510/attachment.pgp>



More information about the ffmpeg-devel mailing list