From: Charles Davis <cdavis5x@gmail.com>
Subject: Re: [PATCH] include: Add dxcore.idl.
Message-Id: <CAPk0b1qR9SrarSnXDnYqS35wsKv=09vmTjbsT39rVfSHgJhSLg@mail.gmail.com>
Date: Fri, 24 Jun 2022 23:30:46 -0500
In-Reply-To: <CALK-3mJ-Cu_apx9=rV6UAbnk1m1OjXqgdQLg=X9bP3_GYq1UTg@mail.gmail.com>
References: <CALK-3mJ-Cu_apx9=rV6UAbnk1m1OjXqgdQLg=X9bP3_GYq1UTg@mail.gmail.com>

On Fri, Jun 24, 2022 at 11:00 PM Biswapriyo Nath <nathbappai@gmail.com> wrote:
> cpp_quote("STDAPI DXCoreCreateAdapterFactory(REFIID riid, void **ppvFactory);")

Generally, we prefer writing out the return type and calling
convention (i.e. HRESULT WINAPI--though I guess technically it should
be HRESULT STDAPICALLTYPE, but the calling convention macros are all
just aliases for __stdcall anyway) to using macros like STDAPI. Did
you copy this directly out of Microsoft's IDL file?

The formatting on the interfaces' attributes (in the square brackets)
and methods is also unusual for Wine.

-- 
Chip