Sean Young wrote:
> + ret = libusb_claim_interface(usb->devhdl, USB_INTERFACE);
> + if (ret != 0) {
> + switch (ret) {
I think it would be quite a lot nicer not to have two almost
completely redundant conditionals right after each other. Change
break to return and break on 0 in the switch.
//Peter
------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60134071&iu=/4140/ostg.clktrk
_______________________________________________
sigrok-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sigrok-devel