<div dir="ltr">Hi Ken, <div><br></div><div>Thanks, I'll make the changes. For the dinput & winmm patch 2/3 & 2/2, should I resubmit them as part of my version 4 of the overall patches even if they have been signed-off on? (although I think only the dinput one was signed off on) I'm not anticipating making any changes to signed-off sorting code patch based on the comments.</div><div><br></div><div>Cheers,</div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 10, 2016 at 3:36 PM, Ken Thomases <span dir="ltr"><<a href="mailto:ken@codeweavers.com" target="_blank">ken@codeweavers.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Jun 30, 2016, at 7:02 PM, David Lawrie <<a href="mailto:david.dljunk@gmail.com">david.dljunk@gmail.com</a>> wrote:<br>
><br>
> Tested on OS X 10.10.5.<br>
><br>
> Signed-off-by: David Lawrie <<a href="mailto:david.dljunk@gmail.com">david.dljunk@gmail.com</a>><br>
> ---<br>
> dlls/winejoystick.drv/joystick_osx.c | 30 ++++++++++++++++++++++++++++--<br>
> 1 file changed, 28 insertions(+), 2 deletions(-)<br>
<br>
</span>This has similar issues as the dinput version, the main one being the pointless "result" variable in get_device_property_long().<br>
<span class="HOEnZb"><font color="#888888"><br>
-Ken<br>
<br>
</font></span></blockquote></div><br></div>