[FFmpeg-devel] rmdec.c: add SIPR codec try #2

Reimar Döffinger Reimar.Doeffinger
Tue Mar 17 20:56:36 CET 2009

On Tue, Mar 17, 2009 at 03:45:51PM -0400, Ronald S. Bultje wrote:
> On Tue, Mar 17, 2009 at 3:21 PM, Reimar D?ffinger
> <Reimar.Doeffinger at gmx.de> wrote:
> > That's simply wrong, for the ((i ^ o) & 1) case you will alternate
> > between i & 1 and not, so you can't move that check out of the loop.
> Well, I calculate the mask per iteration so then it's OK (right?). The
> end result was identical to original code.

If the inner part of the loop works for both even and odd i you wouldn't
need the second if, that much is sure.

More information about the ffmpeg-devel mailing list