[MPlayer-dev-eng] [PATCH] fix to support file:///x:/path

[KO Myung-Hun]

Hi/2.

Alban Bedel wrote:
> On Wed, 20 Feb 2008 01:00:15 -0500
> Rich Felker <dalias at aerifal.cx> wrote:
>
>   
>> On Wed, Feb 20, 2008 at 02:53:24PM +0900, KO Myung-Hun wrote:
>>     
>>> Hi/2.
>>>
>>> Currently, MPlayer supports file://path style. But on Windows and
>>> OS/2, local path is expressed by 'file:///x:/path'. That is, '/' is
>>> prefixed to normal path. So we need to remove '/' before 'x:/path'.
>>>       
>> To my knowledge, file://foo is always a relative (non-/-initial) path,
>> i.e. file://foo means ./foo while file:///foo means /foo, at least in
>> standard urls. I think MPlayer is just wrong here. Special-casing
>> windows drive letter nastiness is not the right fix, at least..
>>     
>
> MPlayer work as you describe, it's not wrong :) Now how the path (ie.
> everything after file://) is to be interpreted is obviously OS
> dependend. So such "path fix" are acceptable imho but they must be under
> #ifdef.
>   

Ok. I've updated the patch as your suggestion.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 1.1.8
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 750 MHz with 512 MB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr


-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: stream_file.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080220/f9cf8df0/attachment.asc>