From: Henri Verbeet Subject: Re: [PATCH] include/d3d9: add missing interfaces Message-Id: Date: Fri, 27 Mar 2020 22:42:15 +0430 In-Reply-To: References: On Tue, 24 Mar 2020 at 19:38, Biswapriyo Nath wrote: > > I seem to have missed this patch getting assigned to me. First, could we please get a more descriptive commit message? That may be easier if you split the patch into several patches each only introducing a single interface. Second, do you happen to have an application that needs these interfaces to build against the Wine headers? If they're for internal Wine usage instead, it may make more sense to introduce them together with the code that uses them. > + STDMETHOD(GetCertificate)(THIS_ UINT CertifacteSize, BYTE *ppCertificate) PURE; Please use reasonable parameter names; e.g., "certificate_size", "certificate". The existing code in that header isn't necessarily a great example in that regard, but we try to improve things.