[PATCH] dinput: WriteEffectToFIle dieffects parameter is const

Ken Thomases ken at codeweavers.com
Mon Sep 21 18:18:13 CDT 2015


On Sep 21, 2015, at 5:50 PM, Nikolay Sivov <bunglehead at gmail.com> wrote:
> 
> On 22.09.2015 0:06, Marcus Meissner wrote:
>> Signed-Off-By: Marcus Meissner <marcus at jet.franken.de>
>> ---
>>  dlls/dinput/device.c         | 4 ++--
>>  dlls/dinput/device_private.h | 4 ++--
>>  include/dinput.h             | 8 ++++----
>>  3 files changed, 8 insertions(+), 8 deletions(-)
>> 
> 
> PSDK header doesn't use const for that argument, I checked with Win10 SDK. Does it really make sense to diverge from it in this case?

I think Marcus misunderstood Elias's earlier email about this.  Elias was pointing out that, while the _docs_ claim the parameter is const, it isn't actually declared that way in Microsoft's headers but is in Wine's headers (for IDirectInputDevice8::WriteEffectToFile()).  Apparently, Marcus thought he was saying it was the other way, so he "corrected" the wrong way.

-Ken




More information about the wine-devel mailing list