[PATCH v2] Change the OSTC3 macros to have a ; in the end

Anton Lundin glance at acc.umu.se
Wed Nov 19 13:36:03 PST 2014


This changes the OSTC3 READ_SETTING/WRITE_SETTING macros into something
that can have a ; in the end which makes some editors auto-indent it better.

Signed-off-by: Anton Lundin <glance at acc.umu.se>
---
 configuredivecomputerthreads.cpp | 121 ++++++++++++++++++++-------------------
 1 file changed, 62 insertions(+), 59 deletions(-)

diff --git a/configuredivecomputerthreads.cpp b/configuredivecomputerthreads.cpp
index f89aa8d..6d6e089 100644
--- a/configuredivecomputerthreads.cpp
+++ b/configuredivecomputerthreads.cpp
@@ -433,36 +433,38 @@ void ReadSettingsThread::run()
 			unsigned char uData[1] = {0};
 
 #define READ_SETTING(_OSTC3_SETTING, _DEVICE_DETAIL) \
-			rc = hw_ostc3_device_config_read(m_data->device, _OSTC3_SETTING, uData, sizeof(uData)); \
-			if (rc == DC_STATUS_SUCCESS) \
-				m_deviceDetails->_DEVICE_DETAIL(uData[0]);
-
-			READ_SETTING(OSTC3_DIVE_MODE, setDiveMode)
-			READ_SETTING(OSTC3_SATURATION, setSaturation)
-			READ_SETTING(OSTC3_DESATURATION, setDesaturation)
-			READ_SETTING(OSTC3_LAST_DECO, setLastDeco)
-			READ_SETTING(OSTC3_BRIGHTNESS, setBrightness)
-			READ_SETTING(OSTC3_UNITS, setUnits)
-			READ_SETTING(OSTC3_SAMPLING_RATE,setSamplingRate)
-			READ_SETTING(OSTC3_SALINITY, setSalinity)
-			READ_SETTING(OSTC3_DIVEMODE_COLOR,setDiveModeColor)
-			READ_SETTING(OSTC3_LANGUAGE, setLanguage)
-			READ_SETTING(OSTC3_DATE_FORMAT, setDateFormat)
-			READ_SETTING(OSTC3_COMPASS_GAIN, setCompassGain)
-			READ_SETTING(OSTC3_SAFETY_STOP, setSafetyStop)
-			READ_SETTING(OSTC3_GF_HIGH, setGfHigh)
-			READ_SETTING(OSTC3_GF_LOW, setGfLow)
-			READ_SETTING(OSTC3_PPO2_MIN, setPpO2Min)
-			READ_SETTING(OSTC3_PPO2_MAX, setPpO2Max)
-			READ_SETTING(OSTC3_FUTURE_TTS, setFutureTTS)
-			READ_SETTING(OSTC3_CCR_MODE, setCcrMode)
-			READ_SETTING(OSTC3_DECO_TYPE, setDecoType)
-			READ_SETTING(OSTC3_AGF_SELECTABLE, setAGFSelectable)
-			READ_SETTING(OSTC3_AGF_HIGH, setAGFHigh)
-			READ_SETTING(OSTC3_AGF_LOW, setAGFLow)
-			READ_SETTING(OSTC3_CALIBRATION_GAS_O2, setCalibrationGas)
-			READ_SETTING(OSTC3_FLIP_SCREEN, setFlipScreen)
-			READ_SETTING(OSTC3_SETPOINT_FALLBACK, setSetPointFallback)
+			do { \
+				rc = hw_ostc3_device_config_read(m_data->device, _OSTC3_SETTING, uData, sizeof(uData)); \
+				if (rc == DC_STATUS_SUCCESS) \
+					m_deviceDetails->_DEVICE_DETAIL(uData[0]); \
+			} while (0)
+
+			READ_SETTING(OSTC3_DIVE_MODE, setDiveMode);
+			READ_SETTING(OSTC3_SATURATION, setSaturation);
+			READ_SETTING(OSTC3_DESATURATION, setDesaturation);
+			READ_SETTING(OSTC3_LAST_DECO, setLastDeco);
+			READ_SETTING(OSTC3_BRIGHTNESS, setBrightness);
+			READ_SETTING(OSTC3_UNITS, setUnits);
+			READ_SETTING(OSTC3_SAMPLING_RATE,setSamplingRate);
+			READ_SETTING(OSTC3_SALINITY, setSalinity);
+			READ_SETTING(OSTC3_DIVEMODE_COLOR,setDiveModeColor);
+			READ_SETTING(OSTC3_LANGUAGE, setLanguage);
+			READ_SETTING(OSTC3_DATE_FORMAT, setDateFormat);
+			READ_SETTING(OSTC3_COMPASS_GAIN, setCompassGain);
+			READ_SETTING(OSTC3_SAFETY_STOP, setSafetyStop);
+			READ_SETTING(OSTC3_GF_HIGH, setGfHigh);
+			READ_SETTING(OSTC3_GF_LOW, setGfLow);
+			READ_SETTING(OSTC3_PPO2_MIN, setPpO2Min);
+			READ_SETTING(OSTC3_PPO2_MAX, setPpO2Max);
+			READ_SETTING(OSTC3_FUTURE_TTS, setFutureTTS);
+			READ_SETTING(OSTC3_CCR_MODE, setCcrMode);
+			READ_SETTING(OSTC3_DECO_TYPE, setDecoType);
+			READ_SETTING(OSTC3_AGF_SELECTABLE, setAGFSelectable);
+			READ_SETTING(OSTC3_AGF_HIGH, setAGFHigh);
+			READ_SETTING(OSTC3_AGF_LOW, setAGFLow);
+			READ_SETTING(OSTC3_CALIBRATION_GAS_O2, setCalibrationGas);
+			READ_SETTING(OSTC3_FLIP_SCREEN, setFlipScreen);
+			READ_SETTING(OSTC3_SETPOINT_FALLBACK, setSetPointFallback);
 
 #undef READ_SETTING
 
@@ -975,35 +977,36 @@ void WriteSettingsThread::run()
 
 			unsigned char data[1] = {0};
 #define WRITE_SETTING(_OSTC3_SETTING, _DEVICE_DETAIL) \
-			data[0] = m_deviceDetails->_DEVICE_DETAIL(); \
-			hw_ostc3_device_config_write(m_data->device, _OSTC3_SETTING, data, sizeof(data));
-
-			WRITE_SETTING(OSTC3_DIVE_MODE, diveMode)
-			WRITE_SETTING(OSTC3_SATURATION, saturation)
-			WRITE_SETTING(OSTC3_DESATURATION, desaturation)
-			WRITE_SETTING(OSTC3_LAST_DECO, lastDeco)
-			WRITE_SETTING(OSTC3_BRIGHTNESS, brightness)
-			WRITE_SETTING(OSTC3_UNITS, units)
-			WRITE_SETTING(OSTC3_SAMPLING_RATE, samplingRate)
-			WRITE_SETTING(OSTC3_SALINITY, salinity)
-			WRITE_SETTING(OSTC3_DIVEMODE_COLOR, diveModeColor)
-			WRITE_SETTING(OSTC3_LANGUAGE, language)
-			WRITE_SETTING(OSTC3_DATE_FORMAT, dateFormat)
-			WRITE_SETTING(OSTC3_COMPASS_GAIN, compassGain)
-			WRITE_SETTING(OSTC3_SAFETY_STOP, safetyStop)
-			WRITE_SETTING(OSTC3_GF_HIGH, gfHigh)
-			WRITE_SETTING(OSTC3_GF_LOW, gfLow)
-			WRITE_SETTING(OSTC3_PPO2_MIN, ppO2Min)
-			WRITE_SETTING(OSTC3_PPO2_MAX, ppO2Max)
-			WRITE_SETTING(OSTC3_FUTURE_TTS, futureTTS)
-			WRITE_SETTING(OSTC3_CCR_MODE, ccrMode)
-			WRITE_SETTING(OSTC3_DECO_TYPE, decoType)
-			WRITE_SETTING(OSTC3_AGF_SELECTABLE, aGFSelectable)
-			WRITE_SETTING(OSTC3_AGF_HIGH, aGFHigh)
-			WRITE_SETTING(OSTC3_AGF_LOW, aGFLow)
-			WRITE_SETTING(OSTC3_CALIBRATION_GAS_O2, calibrationGas)
-			WRITE_SETTING(OSTC3_FLIP_SCREEN, flipScreen)
-			WRITE_SETTING(OSTC3_SETPOINT_FALLBACK, setPointFallback)
+			do { \
+				data[0] = m_deviceDetails->_DEVICE_DETAIL(); \
+				hw_ostc3_device_config_write(m_data->device, _OSTC3_SETTING, data, sizeof(data)); \
+			} while (0)
+
+			WRITE_SETTING(OSTC3_DIVE_MODE, diveMode);
+			WRITE_SETTING(OSTC3_SATURATION, saturation);
+			WRITE_SETTING(OSTC3_DESATURATION, desaturation);
+			WRITE_SETTING(OSTC3_LAST_DECO, lastDeco);
+			WRITE_SETTING(OSTC3_BRIGHTNESS, brightness);
+			WRITE_SETTING(OSTC3_UNITS, units);
+			WRITE_SETTING(OSTC3_SAMPLING_RATE, samplingRate);
+			WRITE_SETTING(OSTC3_SALINITY, salinity);
+			WRITE_SETTING(OSTC3_DIVEMODE_COLOR, diveModeColor);
+			WRITE_SETTING(OSTC3_LANGUAGE, language);
+			WRITE_SETTING(OSTC3_DATE_FORMAT, dateFormat);
+			WRITE_SETTING(OSTC3_COMPASS_GAIN, compassGain);
+			WRITE_SETTING(OSTC3_SAFETY_STOP, safetyStop);
+			WRITE_SETTING(OSTC3_GF_HIGH, gfHigh);
+			WRITE_SETTING(OSTC3_GF_LOW, gfLow);
+			WRITE_SETTING(OSTC3_PPO2_MIN, ppO2Min);
+			WRITE_SETTING(OSTC3_PPO2_MAX, ppO2Max);
+			WRITE_SETTING(OSTC3_FUTURE_TTS, futureTTS);
+			WRITE_SETTING(OSTC3_CCR_MODE, ccrMode);
+			WRITE_SETTING(OSTC3_DECO_TYPE, decoType);
+			WRITE_SETTING(OSTC3_AGF_SELECTABLE, aGFSelectable);
+			WRITE_SETTING(OSTC3_AGF_HIGH, aGFHigh);
+			WRITE_SETTING(OSTC3_AGF_LOW, aGFLow); WRITE_SETTING(OSTC3_CALIBRATION_GAS_O2, calibrationGas);
+			WRITE_SETTING(OSTC3_FLIP_SCREEN, flipScreen);
+			WRITE_SETTING(OSTC3_SETPOINT_FALLBACK, setPointFallback);
 
 #undef WRITE_SETTING
 
-- 
1.9.1



More information about the subsurface mailing list