From 1515bbc91513eb1c32fc099e70e61028236be546 Mon Sep 17 00:00:00 2001 From: Antonio Ospite Date: Wed, 10 Jun 2015 12:42:39 +0200 Subject: [PATCH 1/1] am7xxx: split declaration and initialization of the "transferred" variable This makes it clearer than we mean to initialize the variable to 0 before _each_ libusb_bulk_transfer() call. --- src/am7xxx.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/am7xxx.c b/src/am7xxx.c index 94b01ac..d619f89 100644 --- a/src/am7xxx.c +++ b/src/am7xxx.c @@ -381,8 +381,9 @@ static void trace_dump_buffer(am7xxx_context *ctx, const char *message, static int read_data(am7xxx_device *dev, uint8_t *buffer, unsigned int len) { int ret; - int transferred = 0; + int transferred; + transferred = 0; ret = libusb_bulk_transfer(dev->usb_device, 0x81, buffer, len, &transferred, 0); if (ret != 0 || (unsigned int)transferred != len) { error(dev->ctx, "%s. Transferred: %d (expected %u)\n", @@ -398,10 +399,11 @@ static int read_data(am7xxx_device *dev, uint8_t *buffer, unsigned int len) static int send_data(am7xxx_device *dev, uint8_t *buffer, unsigned int len) { int ret; - int transferred = 0; + int transferred; trace_dump_buffer(dev->ctx, "sending -->", buffer, len); + transferred = 0; ret = libusb_bulk_transfer(dev->usb_device, 0x1, buffer, len, &transferred, 0); if (ret != 0 || (unsigned int)transferred != len) { error(dev->ctx, "%s. Transferred: %d (expected %u)\n", -- 2.1.4