[FFmpeg-devel] [PATCH] Set systems that should support DOS paths in configure
Ramiro Polla
ramiro.polla
Mon Oct 6 18:48:08 CEST 2008
On Mon, Oct 6, 2008 at 1:42 PM, Ramiro Polla <ramiro.polla at gmail.com> wrote:
> Hi,
>
> On Fri, Oct 3, 2008 at 7:58 PM, M?ns Rullg?rd <mans at mansr.com> wrote:
>> "Ramiro Polla" <ramiro.polla at gmail.com> writes:
>>> $subj
> [...]
>>> Index: libavformat/os_support.h
>>> ===================================================================
>>> --- libavformat/os_support.h (revision 15464)
>>> +++ libavformat/os_support.h (working copy)
>>> @@ -35,6 +35,12 @@
>>> # define lseek(f,p,w) _lseeki64((f), (p), (w))
>>> #endif
>>>
>>> +#ifdef HAVE_DOS_PATHS
>>> + #define is_dos_path(path) (path[1]==':')
>>
>> This allows anything as the "drive letter", though maybe that's
>> considered OK.
>
> According to [0], only letters are allowed for the drive letter
> (altough I've seen some program, fsdext2 IIRC, using numbers). Added
> isalpha() for path[0].
>
>>> +#else
>>> + #define is_dos_path(path) (0)
>>
>> Useless ().
>
> Removed.
>
> Also changed from #define to static inline int like ff_network_init().
>
> Ramiro Polla
> [0] http://msdn.microsoft.com/en-us/library/cc232152.aspx
>
And now without the tabs in os_support.h
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dospaths_7.diff
Type: text/x-diff
Size: 2587 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081006/96d42d2d/attachment.diff>
More information about the ffmpeg-devel
mailing list