RFC: OpenAL32.dll thunk -- demacroized

Nick Burns adger44 at hotmail.com
Mon Jan 1 18:03:42 CST 2007


Thanks for the comments --
FIXME -> ERR done
dllmain ret TRUE -> FALSE done
LGPL thingy added

I will resubmit the patch soonish with these changes... (stupid cold might 
have come back....)

Relays can give you the params (and ret values) -- for now i have the traces 
tell you what exts the app is using -- (should it also show params?)

And Thanks for the testing

- Nick

>From: Stefan Dösinger <stefandoesinger at gmx.at>
>To: Nick Burns <adger44 at hotmail.com>
>CC: wine-devel at winehq.com
>Subject: Re: RFC: OpenAL32.dll thunk -- demacroized
>Date: Mon, 1 Jan 2007 22:42:42 +0100
>
>
>Am 01.01.2007 um 12:06 schrieb Nick Burns:
>
>>Here is my OpenAL32.dll thunk demacroized. Sorry this took so long  -- 
>>finally got some time with the break.
>>
>>This is basically the same patch as before -- supports the same  
>>functionality and all -- but no more cool macros (of doom)
>>(~500 lines -> ~1500 lines)
>>
>>I have added as many extensions as I could find to this thunk.
>>(no idea on where these extensions are available)
>>
>>I would like to fix up the extension handling at some point  (basically it 
>>needs to be more like OpenGL)
>>
>>I would like any comments on this patch
>>And would hopefully like to get it in wine
>Some things I noticed
>
>The openal.c header:
>+/* Written by Nick Burns (adger44 at hotmail.com) */
>+/* while sick */
>+/* now demacroized */
>
>I think you should take the usual LGPL header that all the other  files 
>use.
>
>With regard to the traces, I think the usual convention is to write  all 
>the function parameters in the trace. I don't know how well this  can be 
>done, and I imagine that it is quite some work and makes  autogeneration 
>with a script(like opengl) much harder.
>
>If the openal headers are missing you use stubs that just print a  fixme. 
>This is fine, but I think the fixme's should be ERRs. The  DllMain library 
>in that case should maybe return FALSE in that case,  so the app can deal 
>with the lack of openal(or just fail)
>
>I will give the new version another try with Jedi Academy :-)
>
>Stefan
>
>
>





More information about the wine-devel mailing list