[MPlayer-dev-eng] patch to improve the smb://url feature

Matthieu Tourne matthieu.tourne at gmail.com
Sat May 14 02:50:30 CEST 2005


On 5/13/05, Nico Sabbi <nsabbi at tiscali.it> wrote:
> Matthieu Tourne wrote:
> 
> >The reading over samba shares was possible without mounting, but it
> >was impossible to seek in AVIs.
> >the original code didn't use the function pointer in struct stream_st (stream.h)
> >to seek  and to fill the buffer (fill_buffer), and the filling of the
> >buffer was ensured by an ugly switch case instead of using this
> >systematic mechanism.
> >
> >Like it was, all the opening methods of smb://url are in open_stream()
> >(open.c) so I declared my two functions (smb_seek and smb_fill_buffer)
> > in open.c too. But a new file, like stream_smb.c would simplify the
> >code.
> >
> >
> >
> 
> Putting smb specific code in open.c (that is already one of the worst
> files in mplayer)
> is not acceptable IMHO.
> I agree with the rest of your points:  the smb code was badly placed
> and  it should go in
> a file of it's own (stream_smb.c) following the new stream api.
> 
> Will you provide the proper patch? :)
>     Nico

I agree with you, it was mostly too show that there was a problem with
smb handling, and I thought it wasn't a great idea to modify all the
code for my first patch.
I'll do a bigger patch with a cleaner code.

Thank you for your feedback.
Matt.




More information about the MPlayer-dev-eng mailing list