diff options
author | 2015-03-05 17:15:58 +0000 | |
---|---|---|
committer | 2015-03-05 17:15:58 +0000 | |
commit | 9c0ba24f3b54fa7916626f3d76ec6818d78ca47e (patch) | |
tree | c1e568a857341378a9cbdff7ff4ca62722e8801c | |
parent | 686a1b2f4ac90c3fb522405258212c8a91ed5148 (diff) | |
parent | 6695b9920d15f8d9a17d6b0c66b863d1c2e38584 (diff) |
am 6695b992: Merge "Frameworks/base: Add removeAll for ArraySet"
* commit '6695b9920d15f8d9a17d6b0c66b863d1c2e38584':
Frameworks/base: Add removeAll for ArraySet
-rw-r--r-- | core/java/android/util/ArraySet.java | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/core/java/android/util/ArraySet.java b/core/java/android/util/ArraySet.java index 68f725e688bd..7da3941914f6 100644 --- a/core/java/android/util/ArraySet.java +++ b/core/java/android/util/ArraySet.java @@ -475,6 +475,26 @@ public final class ArraySet<E> implements Collection<E>, Set<E> { } /** + * Perform a {@link #remove(Object)} of all values in <var>array</var> + * @param array The array whose contents are to be removed. + */ + public boolean removeAll(ArraySet<? extends E> array) { + // TODO: If array is sufficiently large, a marking approach might be beneficial. In a first + // pass, use the property that the sets are sorted by hash to make this linear passes + // (except for hash collisions, which means worst case still n*m), then do one + // collection pass into a new array. This avoids binary searches and excessive memcpy. + final int N = array.mSize; + + // Note: ArraySet does not make thread-safety guarantees. So instead of OR-ing together all + // the single results, compare size before and after. + final int originalSize = mSize; + for (int i = 0; i < N; i++) { + remove(array.valueAt(i)); + } + return originalSize != mSize; + } + + /** * Return the number of items in this array map. */ @Override |