AP_Generator: avoid use of int16_t-read

ArduPilot · Jul 2, 2024 · 85f8981 · 85f8981
1 parent 42805aa
commit 85f8981
Showing 1 changed file with 3 additions and 5 deletions.
diff --git a/libraries/AP_Generator/AP_Generator_IE_FuelCell.cpp b/libraries/AP_Generator/AP_Generator_IE_FuelCell.cpp
@@ -43,11 +43,9 @@ void AP_Generator_IE_FuelCell::update()
     const uint32_t now = AP_HAL::millis();
 
    // Read any available data
-    uint32_t nbytes = MIN(_uart->available(),30u);
-    while (nbytes-- > 0) {
-        const int16_t c = _uart->read();
-        if (c < 0) {
-            // Nothing to decode
+    for (uint8_t i=0; i<30; i++) {  // process at most n bytes
+        uint8_t c;
+        if (!_uart->read(c)) {
             break;
         }