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

Jason Tackaberry tack at urandom.ca
Wed Aug 19 19:46:37 CEST 2009


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.





More information about the MPlayer-dev-eng mailing list