[PATCH v2 1/2] widl: Define _WIN64 for 64-bit targets.

[Bernhard Übelacker]

Am 23.12.21 um 11:10 schrieb Alistair Leslie-Hughes:
> 
> 
> On 23/12/21 21:05, Bernhard Übelacker wrote:
>> Am 23.12.21 um 09:58 schrieb Alexandre Julliard:
>>> Bernhard Übelacker <bernhardu at mailbox.org> writes:
>>>
>>>> @@ -849,6 +849,8 @@ int main(int argc,char *argv[])
>>>>     add_widl_version_define();
>>>>     wpp_add_cmdline_define("_WIN32=1");
>>>> +  if (pointer_size == 8)
>>>> +    wpp_add_cmdline_define("_WIN64=1");
>>>
>>> You can't use #ifdef _WIN64 in idl files, this would make the generated
>>> headers platform-dependent.
>>>
>>
>> Thanks for looking into it.
>> I guess I could then delay this decision in the idl file like this:
>>
>> cpp_quote("#ifdef _WIN64")
>> cpp_quote("typedef long ADO_LONGPTR;")
> 
> ^^^^^^^^^
> remove the cpp_quote from the above line. That will define it for idl 
> but make it conditional for the compiler.
> 
> Best regards
> Alistair
> 

Thank you, thats it.
While Alexandre has a better way for ADO_LONGPTR, this solves the same 
situation for PositionEnum_Param.