[FFmpeg-devel] [PATCH] Common code for Indeo interactive

Diego Biurrun diego
Fri Mar 20 13:20:27 CET 2009


On Fri, Mar 20, 2009 at 12:25:43PM +0100, Maxim wrote:
> Aurelien Jacobs schrieb:
> > Also your code would benefit from some vertical alignment all over the place,
> > in the code and the data tables.
> >   
> Not sure I understand what you mean but I'll recheck the alignment...

Aurelien is referring to prettyprinting your code.

Things like

    planes[0].width = cfg->pic_width;
    planes[0].height = cfg->pic_height;
    planes[0].num_bands = cfg->luma_bands; 

would look nicer and more readable like this:

    planes[0].width     = cfg->pic_width;
    planes[0].height    = cfg->pic_height;
    planes[0].num_bands = cfg->luma_bands; 

Diego




More information about the ffmpeg-devel mailing list