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

[KO Myung-Hun]

Hi/2.

Diego Biurrun wrote:
> On Mon, Feb 08, 2010 at 11:18:44PM +0900, KO Myung-Hun wrote:
>> Diego Biurrun wrote:
>>> On Mon, Feb 08, 2010 at 09:37:01PM +0900, KO Myung-Hun wrote:
>>>> Diego Biurrun wrote:
>>>>> On Fri, Feb 05, 2010 at 08:53:25PM +0900, KO Myung-Hun wrote:
>>>>>> This patch adds Win32 loader support for OS/2.
>>>>>>
>> I attach the error messages.
>>
>> gcc -MM -Wundef -Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after-statement -std=gnu99 -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -O4 -march=i486 -mtune=generic -pipe -ffast-math -fomit-frame-pointer -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -Ilibdvdread4 -I.   -I/usr/include/freetype2 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer   loader/module.c | sed -e "s,[0-9a-z._-]*: \([a-z0-9/]*/\)[^/]* ,\1&," -e "s,\(.*\)\.o: ,\1.d&,">  loader/module.d
>> gcc -Wundef -Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after-statement -std=gnu99 -Wall -Wno-switch -Wpointer-arith -Wredundant-decls -O4 -march=i486 -mtune=generic -pipe -ffast-math -fomit-frame-pointer -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE -Ilibdvdread4 -I.   -I/usr/include/freetype2 -fno-omit-frame-pointer -mno-omit-leaf-frame-pointer   -c -o loader/module.o loader/module.c
>> In file included from loader/module.c:34:
>> loader/wine/windef.h:115:1: warning: "_stdcall" redefined
>> <built-in>: warning: this is the location of the previous definition
>> loader/wine/windef.h:116:1: warning: "_fastcall" redefined
>> <built-in>: warning: this is the location of the previous definition
>> loader/wine/windef.h:117:1: warning: "__fastcall" redefined
>> <built-in>: warning: this is the location of the previous definition
>> loader/wine/windef.h:124:1: warning: "_cdecl" redefined
>> <built-in>: warning: this is the location of the previous definition
>> loader/wine/windef.h:130:1: warning: "__declspec" redefined
>> <built-in>: warning: this is the location of the previous definition
>
> This sounds like you should adapt the #ifdefs in loader/wine/windef.h.
>

Anyway it's harmless.

>> loader/module.c: In function 'report_func':
>> loader/module.c:896: error: expected expression before 'printf'
>> loader/module.c: At top level:
>> loader/module.c:1052: error: conflicting types for 'MODULE_GetProcAddress'
>> loader/wine/module.h:142: note: previous declaration of 'MODULE_GetProcAddress' was here
>> loader/module.c: In function 'MODULE_GetProcAddress':
>> loader/module.c:1089: error: a label can only be part of a statement and a declaration is not a statement
>> loader/module.c:1089: error: expected identifier or '(' before string constant
>
> The declarations are exactly the same so there must be some trouble
> with different header #includes.
>

The result of #include <os2.h>.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.0
Under OS/2 Warp 4 for Korean with FixPak #15
On Intel Core2Duo T5500 1.66 GHz with 1 GB RAM

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