compile with strict warnings

Even though the code was converting between one enum and the other, it was
continuing to use the old enum value, causing a warning.

Map default: to Undefined so this always works (and never uses an
uninitialized variable).

Change-Id: Ic7da7efebccfb2733498251f67d4d25a56637f44
Sanity-Review: Qt Sanity Bot <qt_sanity_bot@ovi.com>
Reviewed-by: Oliver Wolff <oliver.wolff@nokia.com>

1 files changed, 5 insertions, 5 deletions

diff --git a/src/plugins/sensors/simulator/simulatorambientlightsensor.cpp b/src/plugins/sensors/simulator/simulatorambientlightsensor.cpp
index 040bc2de..0d4a4a4d 100644
--- a/src/plugins/sensors/simulator/simulatorambientlightsensor.cpp
+++ b/src/plugins/sensors/simulator/simulatorambientlightsensor.cpp
@@ -56,9 +56,6 @@ void SimulatorAmbientLightSensor::poll()
     QtMobility::QAmbientLightReadingData data = get_qtAmbientLightData();
     QAmbientLightReading::LightLevel convertedLightLevel;
     switch (data.lightLevel) {
-    case QtMobility::Undefined:
-        convertedLightLevel = QAmbientLightReading::Undefined;
-        break;
     case QtMobility::Dark:
         convertedLightLevel = QAmbientLightReading::Dark;
         break;
@@ -74,6 +71,9 @@ void SimulatorAmbientLightSensor::poll()
     case QtMobility::Sunny:
         convertedLightLevel = QAmbientLightReading::Sunny;
         break;
+    default:
+        convertedLightLevel = QAmbientLightReading::Undefined;
+        break;
     }
 
     quint64 newTimestamp;
@@ -82,9 +82,9 @@ void SimulatorAmbientLightSensor::poll()
     else
         newTimestamp = data.timestamp.toTime_t();
     if (m_reading.timestamp() != newTimestamp
-        || m_reading.lightLevel() != data.lightLevel) {
+        || m_reading.lightLevel() != convertedLightLevel) {
             m_reading.setTimestamp(newTimestamp);
-            m_reading.setLightLevel(data.lightLevel);
+            m_reading.setLightLevel(convertedLightLevel);
 
             newReadingAvailable();
     }