[Live-devel] Re: [vlc-devel] RTSP trickplay support

[Derk-Jan Hartman]

On 20 dec 2004, at 20:57, Ross Finlayson wrote:

> At 11:24 AM 12/20/04, you wrote:
>> On 20 dec 2004, at 19:00, Dermot McGahon wrote:
>>> --- live/liveMedia/MediaSession.cpp     2004-12-15 
>>> 08:31:10.000000000 +0000
>>> +++ live.2004.12.15/liveMedia/MediaSession.cpp  2004-12-20 
>>> 17:56:23.307350100 +0000
>>> @@ -631,6 +631,7 @@
>>>      if (strcmp(fProtocolName, "UDP") == 0) {
>>>        // A UDP-packetized stream (*not* a RTP stream)
>>>        fReadSource = BasicUDPSource::createNew(env(), fRTPSocket);
>>> +      fReadSource = MPEG2TransportStreamFramer::createNew(env(), 
>>> fReadSource);
>>>        fRTPSource = NULL; // Note!
>>>      } else {
>>>        // Check "fCodecName" against the set of codecs that we 
>>> support,
>>
>> This isn't correct. You cannot assume that a UDP source is always 
>> MPEG2 TS. You will need to be sure that there is a MPEG2-TS stream.. 
>> ( payloadtype 33)
>
> No, Dermot's suggested patch was for handling a *raw* UDP stream, 
> *not* a RTP/UDP stream.

Where did I say I assumed otherwise?

> It assumes, however, that all raw UDP streams are MPEG Transport 
> Streams.  Is this likely to be the case in practice?  If not, is there 
> something in the SDP description that will indicate this?  (Does the 
> SDP description for such streams still contain anything useful in the 
> RTP payload format code field (on the "m=" line)?)

Yes the payloadtype...
I have never seen a raw UDP announce that didn't have the correct 
payloadtype set (well early VLC versions :) ).
I don't see the problem....

DJ

> _______________________________________________
> live-devel mailing list
> live-devel at lists.live.com
> http://lists.live.com/mailman/listinfo/live-devel
>
>
---
Universiteit Twente
Derk-Jan Hartman (d.hartman at student.utwente dot nl)
http://home.student.utwente.nl/d.hartman