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

[Alexandre Julliard]

Bernhard Übelacker  writes:
> Am 23.12.21 um 09:58 schrieb Alexandre Julliard:
> > Bernhard Übelacker  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;")
> cpp_quote("#else")
> cpp_quote("typedef __int64 ADO_LONGPTR;")
> cpp_quote("#endif")
>
> But then the type is not known to widl:
>
> .../include/msado15_backcompat.idl:639: error: type 'ADO_LONGPTR' not found in global namespace
>
> How has such a type made known to widl?

LONG_PTR is already defined (as __int3264), you should be able to simply
do 'typedef LONG_PTR ADO_LONGPTR'.