diff options
author | Dirk Hohndel <dirk@hohndel.org> | 2019-03-19 08:22:51 -0700 |
---|---|---|
committer | Dirk Hohndel <dirk@hohndel.org> | 2019-03-19 09:56:48 -0700 |
commit | 8e43d2b8d2023f06e79b1a5b5bfc28bd79294925 (patch) | |
tree | 9a76f381cad875528b0bf0bb7c96fbc62fe1efef /core/configuredivecomputerthreads.cpp | |
parent | c7bb67c5bef01772a9e52dddccfa5f02d91393ff (diff) | |
download | subsurface-8e43d2b8d2023f06e79b1a5b5bfc28bd79294925.tar.gz |
Misc: replace some FIXME comments
All these aren't actually things that need fixing, they are observations about
the code.
Given that LGTM.com reports FIXME comments as Alerts, let's change the ones
that aren't about things that need fixing to something more harmless.
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'core/configuredivecomputerthreads.cpp')
-rw-r--r-- | core/configuredivecomputerthreads.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/core/configuredivecomputerthreads.cpp b/core/configuredivecomputerthreads.cpp index 7ca00b917..9b9374c58 100644 --- a/core/configuredivecomputerthreads.cpp +++ b/core/configuredivecomputerthreads.cpp @@ -1442,7 +1442,7 @@ static dc_status_t read_ostc_settings(dc_device_t *device, DeviceDetails *m_devi unsigned char data[256] = {}; #ifdef DEBUG_OSTC_CF - // FIXME: how should we report settings not supported back? + // open question: how should we report settings not supported back? unsigned char max_CF = 0; #endif rc = hw_ostc_device_eeprom_read(device, 0, data, sizeof(data)); @@ -1829,7 +1829,7 @@ static dc_status_t write_ostc_settings(dc_device_t *device, DeviceDetails *m_dev else if (gas5.type == 2) data[33] = 5; else - // FIXME: No gas was First? + // odd: No gas was First? // Set gas 1 to first data[33] = 1; @@ -1956,7 +1956,7 @@ static dc_status_t write_ostc_settings(dc_device_t *device, DeviceDetails *m_dev else if (dil5.type == 2) data[115] = 5; else - // FIXME: No first diluent? + // odd: No first diluent? // Set gas 1 to fist data[115] = 1; @@ -2178,7 +2178,7 @@ void WriteSettingsThread::run() } break; case DC_FAMILY_HW_OSTC3: - // FIXME: Is this the best way? + // Is this the best way? if (m_deviceDetails->model == "OSTC 4") rc = write_ostc4_settings(m_data->device, m_deviceDetails, DeviceThread::event_cb, this); else |