From 3a4da7c5fa827970a86777ee6f4dc201246f0d0d Mon Sep 17 00:00:00 2001
From: Jean Cyr <jcyr@dillobits.com>
Date: Sat, 26 Jul 2014 09:05:44 -0400
Subject: Revert to original loop

Original loop was correct, and slightly more efficient. Retain
initialization of ret to handle the case where passed in count is 0.
---
 src/drivers/px4io/px4io_uploader.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

(limited to 'src/drivers')

diff --git a/src/drivers/px4io/px4io_uploader.cpp b/src/drivers/px4io/px4io_uploader.cpp
index 986e39dc8..fb16f891f 100644
--- a/src/drivers/px4io/px4io_uploader.cpp
+++ b/src/drivers/px4io/px4io_uploader.cpp
@@ -274,12 +274,11 @@ int
 PX4IO_Uploader::recv_bytes(uint8_t *p, unsigned count)
 {
 	int ret = OK;
-	while (count) {
+	while (count--) {
 		ret = recv_byte_with_timeout(p++, 5000);
 
 		if (ret != OK)
 			break;
-		count--;
 	}
 	return ret;
 }
-- 
cgit v1.2.3