[MPlayer-dev-eng] Re: [PATCH] Repairing breakage to RTP streaming

Ross Finlayson finlayson at live.com
Wed Feb 12 21:31:10 CET 2003


> >  #ifdef STREAMING_LIVE_DOT_COM
> > - case DEMUXER_TYPE_RTP:
> > -   // If the RTP stream is a MPEG stream, then we use this code to check
> > -   // for MPEG headers:
> > -   if (!demux_is_mpeg_rtp_stream(d_video->demuxer)) break;
> > -   // otherwise fall through to...
> > -#endif
> >   case DEMUXER_TYPE_MPEG4_ES: {
> >     videobuf_len=0; videobuf_code_len=0;
>
>wrong!

No, right! :-)

>DEMUXER_TYPE_MPEG4_ES is unrelated to your live.com stuff.
>don't #ifdef it!

Please look at the patch again.  My "#ifdef/#endif" does *not* include the 
DEMUXER_TYPE_MPEG4_ES branch.  The patch simply moves it so that my code 
falls through to the MPEG-1/2 parsing code (as it did originally).

         Ross.



More information about the MPlayer-dev-eng mailing list