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

[Uoti Urpala]

On Wed, 2009-08-19 at 13:46 -0400, Jason Tackaberry wrote:
> On Wed, 2009-08-19 at 19:41 +0200, Diego Biurrun wrote:
> > > +    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?
> 
> It's necessary for those case statements that declare variables,
> otherwise you'd get, as in this one, a "expected expression before
> 'int8_t'" error.

A somewhat less ugly workaround is adding a ";" after the label (the
problem is that the C syntax only allows a label before a statement, and
a declaration is not a statement; with ';' the label is followed by an
empty statement).