[FFmpeg-soc] [PATCH] qdm2 depacketizer

Martin Storsjö martin at martin.st
Wed Jul 14 16:38:22 CEST 2010


On Wed, 14 Jul 2010, Ronald S. Bultje wrote:

> On Wed, Jul 14, 2010 at 6:27 AM, Josh Allmann <joshua.allmann at gmail.com> wrote:
> > That makes the whole hunk tautologous, so simplified to
> > *timestamp = qdm->timestamp.
> >
> > Although the whole thing still feels a bit odd, as we assign
> > qdm->timestamp = *timestamp earlier in the code, and don't use
> > qdm->timestamp otherwise.
> >
> > I also removed the pkt->pts code, as that doesn't appear to be used.
> 
> Can you try removing qdm->timestamp and see what happens? I'm thinking
> I may have added it without there being a good reason...

I tested it, but it's quite needed, otherwise all packets returned get the 
same timestamps, and that's not good.

// Martin


More information about the FFmpeg-soc mailing list