[Ffmpeg-devel] BlackFin lowlevel pixel operations PATCH

Marc Hoffman mmh
Sun Apr 1 22:36:26 CEST 2007


Diego Biurrun writes:
 > On Sun, Apr 01, 2007 at 07:34:04AM -0400, Marc Hoffman wrote:
 > Content-Description: message body text
 > > Marc Hoffman writes:
 > >  > 
 > >  > Diego Biurrun writes:
 > >  >  > On Fri, Mar 30, 2007 at 07:32:14AM -0400, Marc Hoffman wrote:
 > >  >  > > 
 > >  >  > > --- bfin/dsputil_bfin.c	(revision 8517)
 > >  >  > > +++ bfin/dsputil_bfin.c	(working copy)
 > >  >  > > @@ -18,38 +21,296 @@
 > >  >  > >  
 > >  >  > > +static void bfin_idct_add (uint8_t *dest, int line_size, DCTELEM *block)
 > >  >  > > +{
 > >  >  > > +  ff_bfin_idct (block);
 > >  >  > > +  ff_bfin_add_pixels_clamped (block, dest, line_size);
 > >  >  > > +}
 > >  >  > 
 > >  >  > Here and everywhere else: FFmpeg coding style mandates 4 space
 > >  >  > indentation.
 > >  >  > 
 > >  > 
 > >  > No problem, is that for every indentation level or just the first level?
 > >  > 
 > > dsputils indented.
 > 
 > Please use four spaces everywhere.
 > 

I took the time to rewrite the comments and most of the broken english
has been replaced with something a little more useful. Thanks

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bfin.p0
Type: text/x-patch
Size: 62210 bytes
Desc: blackfin dspops
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070401/b55229c4/attachment.bin>



More information about the ffmpeg-devel mailing list