Merge "If the handle does not exist, it should return false" into main

Treehugger Robot · Gerrit Code Review · commit 16fbe775c0fe · 2024-08-30T18:10:37.000Z
diff --git a/services/sensorservice/SensorEventConnection.cpp b/services/sensorservice/SensorEventConnection.cpp
@@ -173,7 +173,7 @@ bool SensorService::SensorEventConnection::addSensor(int32_t handle) {
 
 bool SensorService::SensorEventConnection::removeSensor(int32_t handle) {
     Mutex::Autolock _l(mConnectionLock);
-    if (mSensorInfo.erase(handle) >= 0) {
+    if (mSensorInfo.erase(handle) > 0) {
         return true;
     }
     return false;

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ bool SensorService::SensorEventConnection::addSensor(int32_t handle) {`
`173`	`173`
`174`	`174`	`bool SensorService::SensorEventConnection::removeSensor(int32_t handle) {`
`175`	`175`	`Mutex::Autolock _l(mConnectionLock);`
`176`		`- if (mSensorInfo.erase(handle) >= 0) {`
	`176`	`+ if (mSensorInfo.erase(handle) > 0) {`
`177`	`177`	`return true;`
`178`	`178`	`}`
`179`	`179`	`return false;`