Hi Matthias,
On Thursday, 13 July 2017 18:53:25 CEST Matthias Janke wrote:
> I've appended a more proper patch to fix this behaviour. It extends
> the argument list of get_strings_and_reopen() and get_strings(). I
> introduced default arguments to make this API-break backwards
> compatible. These could, maybe at a later stage, be removed. Whats
> not clear to me is how to deal with open(struct libusb_device *dev).
> Should in here get_strings_and_reopen() not completely removed?
patch has been applied, thanks! I've massaged the changelog
a little and replaced tabs with spaces in two places.
> While this patch tries to minimize the api breakage. I'd suggest to
> join get_strings_and_reopen() and get_strings() and to make them
> private. as they are only useful for vendor(), product() and serial()
> while we should save the used stings in the member variables after
> successfully opening a device based on their usage.
let's postpone any change like that for the next release,
I really like to get this release shipped.
If nobody complains, I'll do a v1.4rc2 next week
and hopefully that will be the final version 1.4, too.
Cheers,
Thomas
--
libftdi - see http://www.intra2net.com/en/developer/libftdi for details.
To unsubscribe send a mail to libftdi+unsubscribe@xxxxxxxxxxxxxxxxxxxxxxx
|