From 2c82a4b593bbf21430f29d7eb894198839b77487 Mon Sep 17 00:00:00 2001 From: Patrick Baumann Date: Thu, 16 Jan 2020 14:56:23 -0800 Subject: Fixes SparseSetArray#add return logic Docs state that adding a key value pair that already exists will return false, else true. The actual behavior is reverse. This change fixes that. Test: atest AppStateTrackerTest Change-Id: I2593f2ba22b5972cccb1c0b3dc5aabb4ed65a023 --- core/java/android/util/SparseSetArray.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'core/java/android/util/SparseSetArray.java') diff --git a/core/java/android/util/SparseSetArray.java b/core/java/android/util/SparseSetArray.java index 9f0f24617f5d..f5025f7a9e99 100644 --- a/core/java/android/util/SparseSetArray.java +++ b/core/java/android/util/SparseSetArray.java @@ -37,10 +37,10 @@ public class SparseSetArray { mData.put(n, set); } if (set.contains(value)) { - return true; + return false; } set.add(value); - return false; + return true; } /** -- cgit v1.2.3