From 766252cd87740fd60023ada09bdc2e02566e0db0 Mon Sep 17 00:00:00 2001 From: Lucas De Marchi Date: Mon, 14 Dec 2015 15:58:00 -0200 Subject: [PATCH] AP_Compass: don't use ternary operator in place of if/else --- libraries/AP_Compass/Compass.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libraries/AP_Compass/Compass.cpp b/libraries/AP_Compass/Compass.cpp index 5f04abbadc..b043ad60b6 100644 --- a/libraries/AP_Compass/Compass.cpp +++ b/libraries/AP_Compass/Compass.cpp @@ -431,8 +431,11 @@ void Compass::_detect_backends(void) // detect_mpu9250() failed will cause panic if no actual mpu9250 backend, // in BH, only one compass should be detected AP_Compass_Backend *backend = AP_Compass_HMC5843::detect_i2c(*this, hal.i2c); - backend ? _add_backend(backend) - : _add_backend(AP_Compass_AK8963::detect_mpu9250(*this, 0)); + if (backend) { + _add_backend(backend); + } else { + _add_backend(AP_Compass_AK8963::detect_mpu9250(*this, 0)); + } #elif CONFIG_HAL_BOARD == HAL_BOARD_LINUX && \ CONFIG_HAL_BOARD_SUBTYPE != HAL_BOARD_SUBTYPE_LINUX_NONE && \ CONFIG_HAL_BOARD_SUBTYPE != HAL_BOARD_SUBTYPE_LINUX_BEBOP && \