[PATCH 1/4] dx8vb: Add stubs for D3DX8 interface

Matteo Bruni matteo.mystral at gmail.com
Tue Oct 31 20:31:06 CDT 2017


2017-10-31 22:30 GMT+01:00 Fabian Maurer <dark.shadow4 at web.de>:
>> (It certainly isn't helpful that
>> https://wiki.winehq.org/Wine_Developer%27s_Guide/COM_in_Wine#Implementing_a_
>> COM_interface. contains loads of old suggestions)
>>
>> >>> +static HRESULT WINAPI d3dx8_QueryInterface(ID3DX8 *iface, REFIID riid,
>> >>> void **ppv) +{
>> >>> +    d3dx8 *This = impl_from_ID3DX8(iface);
>> >>
>> >> Please use lowercase variable names.
>> >
>> > Again, it's like this in 95% of all other cases. I'll change it if you
>> > want, I just wanted it to remain consistent with the existing code.
>>
>> Similarly I can only refer to ddraw/d3d8/9/10/11 here, but other places
>> in Wine certainly differ.
>
> No problem, I'll change that. It's not a problem to use words that are
> reserved words in C++ since we're using only plain C, right?

It's not a problem but in d3d-related code we tend to avoid using
"this" as the name for object variables (e.g. you would name it
"d3dx8" here, or something like that).

In general, please give a look at recent wined3d code / commits to
have an idea WRT the code style we prefer to use in d3d.



More information about the wine-devel mailing list