diff --git a/src/main/io/osd.c b/src/main/io/osd.c index 357b826a5a..2eee3fdc0c 100644 --- a/src/main/io/osd.c +++ b/src/main/io/osd.c @@ -1787,15 +1787,28 @@ static void osdShowStats(uint16_t endBatteryVoltage) osdDisplayStatisticLabel(top++, osdTimerSourceNames[OSD_TIMER_SRC(osdConfig()->timers[OSD_TIMER_2])], buff); } -#ifdef USE_GPS - if (osdStatGetState(OSD_STAT_MAX_SPEED) && featureIsEnabled(FEATURE_GPS)) { - itoa(stats.max_speed, buff, 10); - osdDisplayStatisticLabel(top++, "MAX SPEED", buff); + if (osdStatGetState(OSD_STAT_MAX_ALTITUDE)) { + osdFormatAltitudeString(buff, stats.max_altitude); + osdDisplayStatisticLabel(top++, "MAX ALTITUDE", buff); } - if (osdStatGetState(OSD_STAT_MAX_DISTANCE) && featureIsEnabled(FEATURE_GPS)) { - tfp_sprintf(buff, "%d%c", osdGetMetersToSelectedUnit(stats.max_distance), osdGetMetersToSelectedUnitSymbol()); - osdDisplayStatisticLabel(top++, "MAX DISTANCE", buff); +#ifdef USE_GPS + if (featureIsEnabled(FEATURE_GPS)) { + if (osdStatGetState(OSD_STAT_MAX_SPEED)) { + itoa(stats.max_speed, buff, 10); + osdDisplayStatisticLabel(top++, "MAX SPEED", buff); + } + + if (osdStatGetState(OSD_STAT_MAX_DISTANCE)) { + tfp_sprintf(buff, "%d%c", osdGetMetersToSelectedUnit(stats.max_distance), osdGetMetersToSelectedUnitSymbol()); + osdDisplayStatisticLabel(top++, "MAX DISTANCE", buff); + } + + if (osdStatGetState(OSD_STAT_FLIGHT_DISTANCE)) { + const uint32_t distanceFlown = GPS_distanceFlownInCm / 100; + tfp_sprintf(buff, "%d%c", osdGetMetersToSelectedUnit(distanceFlown), osdGetMetersToSelectedUnitSymbol()); + osdDisplayStatisticLabel(top++, "FLIGHT DISTANCE", buff); + } } #endif @@ -1833,11 +1846,6 @@ static void osdShowStats(uint16_t endBatteryVoltage) } } - if (osdStatGetState(OSD_STAT_MAX_ALTITUDE)) { - osdFormatAltitudeString(buff, stats.max_altitude); - osdDisplayStatisticLabel(top++, "MAX ALTITUDE", buff); - } - #ifdef USE_BLACKBOX if (osdStatGetState(OSD_STAT_BLACKBOX) && blackboxConfig()->device && blackboxConfig()->device != BLACKBOX_DEVICE_SERIAL) { osdGetBlackboxStatusString(buff); @@ -1876,14 +1884,6 @@ static void osdShowStats(uint16_t endBatteryVoltage) } #endif -#ifdef USE_GPS - if (osdStatGetState(OSD_STAT_FLIGHT_DISTANCE) && featureIsEnabled(FEATURE_GPS)) { - const uint32_t distanceFlown = GPS_distanceFlownInCm / 100; - tfp_sprintf(buff, "%d%c", osdGetMetersToSelectedUnit(distanceFlown), osdGetMetersToSelectedUnitSymbol()); - osdDisplayStatisticLabel(top++, "FLIGHT DISTANCE", buff); - } -#endif - #if defined(USE_GYRO_DATA_ANALYSE) if (osdStatGetState(OSD_STAT_MAX_FFT) && featureIsEnabled(FEATURE_DYNAMIC_FILTER)) { int value = getMaxFFT(); @@ -1895,7 +1895,6 @@ static void osdShowStats(uint16_t endBatteryVoltage) } } #endif - } static void osdShowArmed(void) diff --git a/src/test/unit/osd_unittest.cc b/src/test/unit/osd_unittest.cc index 23948be217..a9dd93da5a 100644 --- a/src/test/unit/osd_unittest.cc +++ b/src/test/unit/osd_unittest.cc @@ -394,13 +394,13 @@ TEST(OsdTest, TestStatsImperial) displayPortTestBufferSubstring(2, row++, "2017-11-19 10:12:"); displayPortTestBufferSubstring(2, row++, "TOTAL ARM : 00:05.00"); displayPortTestBufferSubstring(2, row++, "LAST ARM : 00:03"); + displayPortTestBufferSubstring(2, row++, "MAX ALTITUDE : 6.5%c", SYM_FT); displayPortTestBufferSubstring(2, row++, "MAX SPEED : 17"); displayPortTestBufferSubstring(2, row++, "MAX DISTANCE : 328%c", SYM_FT); + displayPortTestBufferSubstring(2, row++, "FLIGHT DISTANCE : 656%c", SYM_FT); displayPortTestBufferSubstring(2, row++, "MIN BATTERY : 14.70%c", SYM_VOLT); displayPortTestBufferSubstring(2, row++, "END BATTERY : 15.20%c", SYM_VOLT); displayPortTestBufferSubstring(2, row++, "MIN RSSI : 25%%"); - displayPortTestBufferSubstring(2, row++, "MAX ALTITUDE : 6.5%c", SYM_FT); - displayPortTestBufferSubstring(2, row++, "FLIGHT DISTANCE : 656%c", SYM_FT); } /* @@ -447,13 +447,13 @@ TEST(OsdTest, TestStatsMetric) displayPortTestBufferSubstring(2, row++, "2017-11-19 10:12:"); displayPortTestBufferSubstring(2, row++, "TOTAL ARM : 00:07.50"); displayPortTestBufferSubstring(2, row++, "LAST ARM : 00:02"); + displayPortTestBufferSubstring(2, row++, "MAX ALTITUDE : 2.0%c", SYM_M); displayPortTestBufferSubstring(2, row++, "MAX SPEED : 28"); displayPortTestBufferSubstring(2, row++, "MAX DISTANCE : 100%c", SYM_M); + displayPortTestBufferSubstring(2, row++, "FLIGHT DISTANCE : 100%c", SYM_M); displayPortTestBufferSubstring(2, row++, "MIN BATTERY : 14.70%c", SYM_VOLT); displayPortTestBufferSubstring(2, row++, "END BATTERY : 15.20%c", SYM_VOLT); displayPortTestBufferSubstring(2, row++, "MIN RSSI : 25%%"); - displayPortTestBufferSubstring(2, row++, "MAX ALTITUDE : 2.0%c", SYM_M); - displayPortTestBufferSubstring(2, row++, "FLIGHT DISTANCE : 100%c", SYM_M); } /*