[FFmpeg-soc] vp8 de/packetizers

Martin Storsjö martin at martin.st
Thu Aug 12 21:56:28 CEST 2010


On Thu, 12 Aug 2010, Luca Abeni wrote:

> On 12/08/10 16:07, Martin Storsjö wrote:
> > On Wed, 11 Aug 2010, Josh Allmann wrote:
> > 
> > > > The packetizer looks ok to me. Perhaps still adding a warning message at
> > > > startup that the spec isn't finalized yet and may still change, as Luca
> > > > A
> > > > pointed out?
> > > > 
> > > 
> > > Done. The best place to print this was in sdp.c. Also added a similar
> > > note when initializing the depacketizer.
> > 
> > The packetizer looks good to me now, do you agree, Luca A?
> Sorry, I think I did not receive the patch... I'll have a look at the
> ffmpeg-soc archives.
> Anyway, I think the warning should be printed with av_log() (printing it in
> the SDP as a comment might be an idea, but I'd like to see a warning on stderr
> too).

Yeah, it is printed with av_log, he just prints it in sdp.c instead of 
adding a separate VP8-specific case in the rtpenc initialization (where 
none is needed otherwise).

// Martin


More information about the FFmpeg-soc mailing list