<div dir="ltr">I am compiling with <span style="font-size:13px">-Wdeclaration-after-statement but oddly enough the compiler did not generate that warning for dinput. However, I don't know why not, because I see what you're saying and will make the necessary change anyway.</span><div><span style="font-size:13px"><br></span></div><div><span style="font-size:13px">David</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 10, 2016 at 3:33 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>
> @@ -490,10 +490,10 @@ static int find_osx_devices(void)<br>
>     if (devset)<br>
>     {<br>
>         CFIndex num_devices, num_main_elements, idx;<br>
> -        CFMutableArrayRef devices = CFArrayCreateMutable(kCFAllocatorDefault, 0, &kCFTypeArrayCallBacks);<br>
> -        CFSetApplyFunction(devset, CFSetApplierFunctionCopyToCFArray, devices);<br>
> -        CFRelease( devset);<br>
> -        num_devices = CFArrayGetCount(devices);<br>
> +        num_devices = CFSetGetCount(devset);<br>
> +        CFMutableArrayRef devices = CFArrayCreateMutable(kCFAllocatorDefault, num_devices, &kCFTypeArrayCallBacks);<br>
<br>
</span>You've now interleaved statements and declarations.  Wine's code standard requires that declarations all come before statements.  So, you could change the assignment of num_devices to an initialization or split the declaration and initialization of devices to separate declaration and assignment.<br>
<br>
Wine should be using -Wdeclaration-after-statement for its compile commands.  Therefore, this change would have added a new warning.  I know that some of the Mac files have a lot of warnings, which makes it hard to see new ones, but you should strive to avoid adding more.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Ken<br>
<br>
</font></span></blockquote></div><br></div>