[FFmpeg-devel] [Fwd: Summer of code small task patch]

Diego Biurrun diego
Fri Mar 27 10:08:25 CET 2009


On Wed, Mar 25, 2009 at 05:43:05PM +0200, Dylan Yudaken wrote:
> 
> #include <libavutil/mathematics.h>

This is not a system header, use "" instead of <>.

> /*
>   This file could be optimised a lot, but is for
>   reference and so readability is better
>  */

This comment should be at the top of the file, not intermingled with the
code.

> /**
>  * Initialise the Double Precision Discrete Cosine Transform
>  * functions fdct & idct
>  *
>  */

nits: End sentences in a period, leave out the pointless blank line and
use American English spelling.

Diego



More information about the ffmpeg-devel mailing list