[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