[Live-devel] RTCPInstance and RTP timestamp

[David BERTRAND]

Ross,
Thank yo ufor including this in next release. However, I would like to update slightly my patch. Actually, previous patch didn't prevent from sending the RTCP packet, just omitted the report inside. This lead to a source description RTCP packet, which IMO is not a good idea. Therefore , I now make my check (for a RTP packet being sent firstly) in sendReport() i.s.o addReport() :

void RTCPInstance::sendReport() {

  if (fSink != NULL && !(fSink->haveComputedFirstTimestamp())) {
    return;
  }

#ifdef DEBUG
  fprintf(stderr, "sending REPORT\n");
#endif  
[...]

Sorry for publishing this fix in two times.
David
> ----- Original Message -----
> From: "Ross Finlayson" <finlayson at live555.com>
> To: "LIVE555 Streaming Media - development & use" <live-devel at ns.live555.com>
> Subject: Re: [Live-devel] RTCPInstance and RTP timestamp
> Date: Wed, 19 Jul 2006 05:07:55 -0700
> 
> 
> > I recognize my problem is quite rare (if you stream from a file, 
> > you can send RTP packets immediately, so RTCP packets will 
> > probably be sent after the first RTP packets) and also very 
> > difficult to resolve properly. IMO, the only acceptable way is to 
> > send RTCP packets only if a RTP packet has already been sent for 
> > the same track. This fix works for my particular problem and I 
> > would like to propose it as patch for the library.
> 
> That looks reasonable.  It will be included in the next release of
> the software.
> --
> Ross Finlayson
> Live Networks, Inc.
> http://www.live555.com/
> _______________________________________________
> live-devel mailing list
> live-devel at lists.live555.com
> http://lists.live555.com/mailman/listinfo/live-devel

>


-- 
_______________________________________________
Surf the Web in a faster, safer and easier way:
Download Opera 9 at http://www.opera.com

Powered by Outblaze