summaryrefslogtreecommitdiff
path: root/java/com/android/dialer/phonelookup
diff options
context:
space:
mode:
authorzachh <zachh@google.com>2017-11-10 16:29:49 -0800
committerZachary Heidepriem <zachh@google.com>2017-11-11 20:25:54 -0800
commit4c7d959c3e944963436705340eccf4b82cde7e47 (patch)
treec1418f4359ef55468095e627f301bb279ba14d26 /java/com/android/dialer/phonelookup
parent37d5529924979c13c8810f8baa47e6a746b9cc81 (diff)
Use lambdas in CompositePhoneLookup.
They should work now that we target guava 20 instead of 18 in AOSP. Test: tap PiperOrigin-RevId: 175354039 Change-Id: Id7844c3a1c8e29e5ecb13fa36a92dd80be0cfc7c
Diffstat (limited to 'java/com/android/dialer/phonelookup')
-rw-r--r--java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java49
1 files changed, 19 insertions, 30 deletions
diff --git a/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java b/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java
index f85b357e7..59a845774 100644
--- a/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java
+++ b/java/com/android/dialer/phonelookup/composite/CompositePhoneLookup.java
@@ -23,7 +23,6 @@ import com.android.dialer.common.LogUtil;
import com.android.dialer.common.concurrent.DialerFutures;
import com.android.dialer.phonelookup.PhoneLookup;
import com.android.dialer.phonelookup.PhoneLookupInfo;
-import com.google.common.base.Function;
import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
@@ -60,15 +59,12 @@ public final class CompositePhoneLookup implements PhoneLookup {
}
return Futures.transform(
Futures.allAsList(futures),
- new Function<List<PhoneLookupInfo>, PhoneLookupInfo>() {
- @Override
- public PhoneLookupInfo apply(List<PhoneLookupInfo> infos) {
- PhoneLookupInfo.Builder mergedInfo = PhoneLookupInfo.newBuilder();
- for (PhoneLookupInfo info : infos) {
- mergedInfo.mergeFrom(info);
- }
- return mergedInfo.build();
+ infos -> {
+ PhoneLookupInfo.Builder mergedInfo = PhoneLookupInfo.newBuilder();
+ for (PhoneLookupInfo info : infos) {
+ mergedInfo.mergeFrom(info);
}
+ return mergedInfo.build();
},
MoreExecutors.directExecutor());
}
@@ -102,30 +98,23 @@ public final class CompositePhoneLookup implements PhoneLookup {
}
return Futures.transform(
Futures.allAsList(futures),
- new Function<
- List<ImmutableMap<DialerPhoneNumber, PhoneLookupInfo>>,
- ImmutableMap<DialerPhoneNumber, PhoneLookupInfo>>() {
- @Override
- public ImmutableMap<DialerPhoneNumber, PhoneLookupInfo> apply(
- List<ImmutableMap<DialerPhoneNumber, PhoneLookupInfo>> allMaps) {
- ImmutableMap.Builder<DialerPhoneNumber, PhoneLookupInfo> combinedMap =
- ImmutableMap.builder();
- for (DialerPhoneNumber dialerPhoneNumber : existingInfoMap.keySet()) {
- PhoneLookupInfo.Builder combinedInfo = PhoneLookupInfo.newBuilder();
- for (ImmutableMap<DialerPhoneNumber, PhoneLookupInfo> map : allMaps) {
- PhoneLookupInfo subInfo = map.get(dialerPhoneNumber);
- if (subInfo == null) {
- throw new IllegalStateException(
- "A sublookup didn't return an info for number: "
- + LogUtil.sanitizePhoneNumber(
- dialerPhoneNumber.getRawInput().getNumber()));
- }
- combinedInfo.mergeFrom(subInfo);
+ (allMaps) -> {
+ ImmutableMap.Builder<DialerPhoneNumber, PhoneLookupInfo> combinedMap =
+ ImmutableMap.builder();
+ for (DialerPhoneNumber dialerPhoneNumber : existingInfoMap.keySet()) {
+ PhoneLookupInfo.Builder combinedInfo = PhoneLookupInfo.newBuilder();
+ for (ImmutableMap<DialerPhoneNumber, PhoneLookupInfo> map : allMaps) {
+ PhoneLookupInfo subInfo = map.get(dialerPhoneNumber);
+ if (subInfo == null) {
+ throw new IllegalStateException(
+ "A sublookup didn't return an info for number: "
+ + LogUtil.sanitizePhoneNumber(dialerPhoneNumber.getRawInput().getNumber()));
}
- combinedMap.put(dialerPhoneNumber, combinedInfo.build());
+ combinedInfo.mergeFrom(subInfo);
}
- return combinedMap.build();
+ combinedMap.put(dialerPhoneNumber, combinedInfo.build());
}
+ return combinedMap.build();
},
MoreExecutors.directExecutor());
}