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
_______________________________________________
FFmpeg-soc mailing list
FFmpeg-soc@mplayerhq.hu
https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-soc

Reply via email to