<div dir="ltr">Right-o! I was wondering what the CFRelease was doing in the debugstr_cf function. :) <div><br></div><div>Cheers,</div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 17, 2016 at 2:11 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 Jul 17, 2016, at 4:08 PM, Ken Thomases <<a href="mailto:ken@codeweavers.com">ken@codeweavers.com</a>> wrote:<br>
><br>
> On Jul 16, 2016, at 3:15 AM, David Lawrie <<a href="mailto:david.dljunk@gmail.com">david.dljunk@gmail.com</a>> wrote:<br>
>><br>
</span><span class="">>> +static CFStringRef get_device_name(IOHIDDeviceRef device)<br>
>> +{<br>
>> +    CFTypeRef ref;<br>
>> +    CFStringRef name = CFSTR("Default Name");<br>
><br>
> You can probably do better for a fallback.  For example, you can format the vendor ID and product ID into a string.  For an extreme take on this, see the Copy_DeviceName() function in Apple's HID Calibrator sample code:<br>
> <a href="https://developer.apple.com/library/mac/samplecode/HID_Calibrator/Listings/HID_Calibrator_IOHIDDeviceWindowCtrl_m.html" rel="noreferrer" target="_blank">https://developer.apple.com/library/mac/samplecode/HID_Calibrator/Listings/HID_Calibrator_IOHIDDeviceWindowCtrl_m.html</a><br>
<br>
</span>Oh, and for the case where device is NULL, it's fine to return NULL.  It should never happen and, if it does, we want a crash to figure out why and fix things in a better manner, rather than papering over it with a bogus value.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Ken<br>
<br>
</font></span></blockquote></div><br></div>