[MPlayer-dev-eng] [PATCH] Fix reordering for 24-bit sample content

Reimar Döffinger Reimar.Doeffinger at gmx.de
Wed Aug 19 19:45:02 CEST 2009


On Wed, Aug 19, 2009 at 07:41:44PM +0200, Diego Biurrun wrote:
> On Wed, Aug 19, 2009 at 09:32:32AM -0400, Jason Tackaberry wrote:
> > 
> > --- mplayer/libaf/reorder_ch.c	(revision 29541)
> > +++ mplayer/libaf/reorder_ch.c	(working copy)
> > @@ -189,6 +189,94 @@
> >  
> > +    int i;
> > +    switch (samplesize) {
> > +    case 1:
> > +    {
> > +        int8_t *dest_8 = dest;
> > +        const int8_t *src_8 = src;
> > +        REORDER_COPY_8(dest_8,src_8,samples,s0,s1,s2,s3,s4,s5,s6,s7);
> > +        break;
> > +    }
> 
> This is probably a stupid question but anyway: Are the {} in the case
> statements necessary?

For gcc 2.95 yes.



More information about the MPlayer-dev-eng mailing list