[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