[MPlayer-cvslog] r32731 - in trunk: path.c path.h

[KO Myung-Hun]

Hi/2.

Reimar Döffinger wrote:
> On 26 dec 2010, at 09:22, KO Myung-Hun <komh78 at gmail.com> wrote:
>> Clément Bœsch wrote:
>>> On Sun, Dec 26, 2010 at 02:12:19AM +0900, KO Myung-Hun wrote:
>>>> Hi/2.
>>>>
>>>> cboesch wrote:
>>>>> Author: cboesch
>>>>> Date: Sat Dec 25 00:09:08 2010
>>>>> New Revision: 32731
>>>>>
>>>>> Log:
>>>>> Add mp_path_is_absolute function.
>>>>>
>>>>> Modified:
>>>>>   trunk/path.c
>>>>>   trunk/path.h
>>>>>
>>>>> Modified: trunk/path.c
>>>>> ==============================================================================
>>>>> --- trunk/path.c    Sat Dec 25 00:05:49 2010    (r32730)
>>>>> +++ trunk/path.c    Sat Dec 25 00:09:08 2010    (r32731)
>>>>> @@ -233,3 +233,15 @@ char *mp_dirname(const char *path)
>>>>>     dirname[len] = '\0';
>>>>>     return dirname;
>>>>> }
>>>>> +
>>>>> +/**
>>>>> + * @brief Indicates weither the path is absolute or not.
>>>>> + */
>>>>> +int mp_path_is_absolute(const char *path)
>>>>> +{
>>>>> +#if HAVE_DOS_PATHS
>>>>> +    return path[0] && path[1] == ':';
>>>>> +#else
>>>>> +    return path[0] == '/';
>>>>> +#endif
>>>>> +}
>>>>>
>>>>
>>>> You should consider the case that path[0] is '/' or '\\' as well in
>>>> HAVE_DOS_PATHS
>>>>
>>>
>>> Mmmh ok, I'll fix that. But I wonder how opendir() will behave in those
>>> cases (I don't have any windows machine to test).
>>>
>>
>> It uses a current drive.
> 
> Which is what makes this complicated, /something and \something are _not_ absolute paths on windows, they do depend on the current directory.

If so, are you saying that it will be a x:\current_dir\something when
creating \something on Windows ?

It will be x:\something.

Are you confused with ./something and .\something ?

> Neither is c: btw., only c:/ and c:\ and \\something are absolute paths.

But PathIsRelative() return 0 in case of \something, c: and c:\.

I think, the path which can be appended to 'x:\dir\' is relative,
otherwise absolute.


-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0.11
Under OS/2 Warp 4 for Korean with FixPak #15
On AMD ThunderBird 1GHz with 512 MB RAM

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