[RFC] Work done so far

Joshua Joseph joejoshw at gmail.com
Fri May 30 02:21:49 PDT 2014


On Fri, May 30, 2014 at 11:47 AM, Jef Driesen <jef at libdivecomputer.org>
wrote:

> On 2014-05-30 09:52, Joshua Joseph wrote:
>
>> Please check and see if I am on the right track.
>>
>
> I quickly checked the patched, and noticed one libdivecomputer related
> mistake: you shouldn't use the vendor or product strings to check the
> device type. There are much better ways to do this. The strings are mainly
> intended for display in the user interface.
>
> The solution to both of these problems, is to check the family type
> instead, using the dc_device_get_type or dc_descriptor_get_type function:
>
> if (dc_device_get_type(data->device) == DC_FAMILY_HW_FROG) {
>    ...
>    hw_frog_device_version(data->device, hw_data, 10);
>    ...
> }
>
> Jef
>

Thanks, Jef.

I will apply this correction.

-- 
Thanks,
Joshua
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20140530/171126ea/attachment.html>


More information about the subsurface mailing list