[MPlayer-dev-eng] MMS support in mplayer begun

[Bertrand Baudet]

I didn't try the patch yet, I just read it. And I have few other comments:
- Please create a file called asf_mmst_streaming_start(). So you will not
  have to change the asf_http_streaming_start() function.
  You will have also to create an asf_mmst_streaming_read() function,
  that desencapsulate the incoming stream.
  With those functions, it will be much more easier for me to integrate it.
  You will have to call your asf_mmst_streaming_start() from the
  asf_streaming_start function. Just comment the return line and
  put yours for testing.
  The asf_streaming_start function is supposed to select the appropriate
  network streaming type. This function should first try mmsu, then mmst
  and then if they fail(because of firewall/proxy) it should try http.
  For now, what I can do, is having a simple switch on the protocol.
  I want keep mms doing http for now, so you can take the mmst for
  you test. I'm gonna add the switch in this function this week end.
  And I'm also gonna add the mmst entry in the network.c file.

- Please try to not unindent my code, if some code are no indented and
  stick to the left, is because it's temporary and for debuging. It's easier
  for me to find this code when it's not indented.

- Why are you changing the open.c file? I don't understand why you are
  trying to pass the address of the stream pointer? In addition the I don't
  see changes for the streaming_start declaration, so I will break the
  current implementation.

See below for more...

On Thursday 06 December 2001 11:28 am, you wrote:
> BTW, I only currently have my section of code parsing URLs (like
> mms://server/blah). The HTTP code was modified by me, but this patch is
> not intended to make it to CVS yet. The functionality of HTTP code still
> works without any additional changes.
>
> Currently, the format for the URLs (mplayer mms://server/blah). I just
> have my section making an initial connection and returning the servers
> initial response.  The 4.1.3865 indicates it likes the player version.
> I had to set the player version to 6.4.1 to keep the stream (and the
> server) I'm testing happy.

I don't quite understand. Are you sending an HTTP request? For what I know,
mmst doesn't need a HTTP handshake. So please make you code cleaner
according to what I told you at the beginning of this email and in my
previous emails. Thanks.

Bertrand

> Derek J Witt wrote:
> > This section of code is only for MMS tcp support. As Bertrand has said,
> > he is working on pre-buffering code.
> >
> > I do know how that currently the ASX parser  is not finished yet.
> >
> > Hetz Ben-Hamo wrote:
> > > Small question:
> > >
> > > what is the format to use it on mplayer?
> > >
> > > will, for example the following command work?
> > > mplayer -vo xv http://entertainment.msn.com/hs/ted.112901_300.asx
> > >
> > > Also, is there some buffering support? if so, what is the parameter for
> > > that?
> > >
> > > Thanks,
> > > Hetz Ben Hamo
> > > hetz at kde.org
> > >
> > > On Thursday 06 December 2001 11:21 am, you wrote:
> > > > Hi, all. I have started on MMS support in mplayer.  I have it at
> > > > least connecting to TCP MMS streams.  I have included the patch. For
> > > > some reason, if I try using "cvs rdiff 120601 main", I keep getting a
> > > > whole messful of parsed_root_directory errors. So,  I  was only able
> > > > to run "cvs diff -NR" to get a file. I had to edit the file to get a
> > > > proper patch file. But, it didn't include the new file I put in.
> > > >
> > > > So, I have included both the patch and the new file.  I do have my
> > > > test mms stream being connected.  I do send it the correct header. I
> > > > have yet to get it to start the actual data packets. But, it should
> > > > not affect anything else in mplayer. Let me know what you think.
> > > >
> > > > Thanks.
> > >
> > > _______________________________________________
> > > MPlayer-dev-eng mailing list
> > > MPlayer-dev-eng at mplayerhq.hu
> > > http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
> >
> > --
> > **  Derek J Witt                                              **
> > *   Email: mailto:djw at flinthills.com                           *
> > *   Home Page: http://www.flinthills.com/~djw/                 *
> > *** "...and on the eighth day, God met Bill Gates." - Unknown **
> > _______________________________________________
> > MPlayer-dev-eng mailing list
> > MPlayer-dev-eng at mplayerhq.hu
> > http://mplayerhq.hu/mailman/listinfo/mplayer-dev-eng