[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