On Thursday, January 13, 2011 07:55:06 ian wrote:
> The patch is inline below.

unfortunately, it looks like it has been corrupted by your client.  line 
wrapped and excess spaces.

> Please note that 'interface' appeared to be a
> reserved keyword in the Windows compile, so we used interfaceb.

how does libftdi work then ?  it too uses "interface".

> +                if (params[i]->value.lu == 0) {
> +                    p->interfaceb = 0;
> +                } else {
> +                    p->interfaceb = params[i]->value.lu - 1;
> +                }

drop the braces completely

> +                if (ftdi_set_interface(fc, params[i]->value.lu) == -1)
> +                {
> +                    usbconn_ftdi_free(c);

need spaces before the open paren

otherwise it looks fine
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

------------------------------------------------------------------------------
Protect Your Site and Customers from Malware Attacks
Learn about various malware tactics and how to avoid them. Understand 
malware threats, the impact they can have on your business, and how you 
can protect your company and customers by using code signing.
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
UrJTAG-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/urjtag-development

Reply via email to