Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 8d94d82a authored by Mathias Agopian's avatar Mathias Agopian
Browse files

fix [4025681] continuous sensors should not try to send an event as soon as they're activated 

Make sure to send an event down only for sensors that report a value only on data
change. Other sensors, will naturally send an event when the next event is available.

Bug: 4025681
Change-Id: I6d444deda388b6bc9a33e3371e09d390f1566ec5
parent 3de6a9b6

services/sensorservice/SensorService.cpp

+8 −5
Original line number Diff line number Diff line
@@ -340,7 +340,9 @@ status_t SensorService::enable(const sp<SensorEventConnection>& connection, 
            if (rec->addConnection(connection)) {

                // this sensor is already activated, but we are adding a

                // connection that uses it. Immediately send down the last

                // known value of the requested sensor.

                // known value of the requested sensor if it's not a

                // "continuous" sensor.

                if (sensor->getSensor().getMinDelay() == 0) {

                    sensors_event_t scratch;

                    sensors_event_t& event(mLastEventSeen.editValueFor(handle));

                    if (event.version == sizeof(sensors_event_t)) {
@@ -348,6 +350,7 @@ status_t SensorService::enable(const sp<SensorEventConnection>& connection, 
                    }

                }

            }

        }

        if (err == NO_ERROR) {

            // connection now active

            if (connection->addSensor(handle)) {