[MPlayer-dev-eng] [PATCH] mmap support for OS/2

[KO Myung-Hun]

Hi/2.

Diego Biurrun wrote:
> On Mon, Feb 25, 2008 at 07:51:44PM +0900, KO Myung-Hun wrote:
>   
>> --- osdep/mmap.h	(revision 0)
>> +++ osdep/mmap.h	(revision 0)
>> @@ -0,0 +1,56 @@
>> +/*
>> + * mmap.h : mmap declaration header for sys/mman.h missing system
>>     
>
> mmap declaration header for systems missing sys/mman.h
>
>   

Ok.

>> --- osdep/mmap-os2.c	(revision 0)
>> +++ osdep/mmap-os2.c	(revision 0)
>> @@ -0,0 +1,212 @@
>> +/*
>> + * mmap-os2.c : Very simple implementation of mmap function for OS/2
>>     
>
> very simple implementation of mmap() for OS/2
>
>   

Ok.

>> +struct os2_mmap_s;
>> +typedef struct os2_mmap_s
>> +{
>> +    void    *addr;
>> +    size_t  len;
>> +    int     flags;
>> +    struct os2_mmap_s *prev;
>> +    struct os2_mmap_s *next;
>> +} os2_mmap;
>>     
>
>   
>> +static os2_mmap *_mmap = NULL;
>>     
>
> Identifiers starting with _ and lowercase are reserved at the file
> level.  You are not using it outside this file, but it's still ugly.
>
>   

Ok. I prepend 'm'.

>> +        // Allocate tiled memory compatible with 16-bit selectors
>> +        // 'fs_seg' in 'ldt_keeper.c' need this attr
>>     
>
> attribute
>
>   

Ok.

-- 
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: mmap.diff
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20080227/2688e800/attachment.txt>