[MPlayer-dev-eng] [PATCH] processing "file:///" URL (resending)

[Eugene Crosser]

Nico Sabbi wrote:

>> a while ago, I sent a patch that allows mplayer to accept "file:///..."
>> URLs
[...]
> the current stream API provides a mechanism to access urls: in
> stream_file.c
> there's this initilizer
> 
> stream_info_t stream_info_file = {
>  "File",
>  "file",
>  "Albeu",
>  "based on the code from ??? (probably Arpi)",
>  open_f,
>  { "file", "", NULL },
>  &stream_opts,
>  1 // Urls are an option string
> };
> 
> that provides a way to access a file using file:// syntax, but it's not
> working.
> This means there's a bug to fix properly in the stream driver

The problem is that this nice magic is used in open_stream_full(), and
open_stream_full is called from open.c *only* for non-url arguments,
after all the cruft between lines 470-570 of open.c.  So actually the
whole thing with auto_open_streams[] plugins, as far as I understand, is
never used!

Maybe all that code can be removed from open.c, and open_stream_full()
just called blindly for any argument, whether path or url...  But that
probably should be done by someone who wrote that code (or at least
*read* it;)

In the light of the above, what would you suggest to do?

Eugene
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 256 bytes
Desc: OpenPGP digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20050514/04bb68a7/attachment.pgp>