From 751eda886bbc31ed04d13fba9cc40baadcefb175 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rub=C3=A9n=20Porras=20Campo?= <pcr@avaloq.com>
Date: Wed, 4 Dec 2024 06:37:50 +0100
Subject: [PATCH] Remove duplicated access to a set element and use lambas

---
 .../ddk/xtext/findrefs/ReferenceFinder2.java      |  3 +--
 .../extensions/ResourceDescriptionsUtil.java      | 15 ++-------------
 2 files changed, 3 insertions(+), 15 deletions(-)

diff --git a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/findrefs/ReferenceFinder2.java b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/findrefs/ReferenceFinder2.java
index ae82ec50a..f461698d5 100644
--- a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/findrefs/ReferenceFinder2.java
+++ b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/findrefs/ReferenceFinder2.java
@@ -94,8 +94,7 @@ protected List<IReferenceDescription> uniqueReferences(final List<IReferenceDesc
     List<IReferenceDescription> noRepeats = Lists.newArrayListWithExpectedSize(refs.size());
     for (IReferenceDescription ref : refs) {
       String sourceURI = ref.getSourceEObjectUri().toString();
-      if (!addedRefs.contains(sourceURI)) {
-        addedRefs.add(sourceURI);
+      if (addedRefs.add(sourceURI)) {
         noRepeats.add(ref);
       }
     }
diff --git a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/resource/extensions/ResourceDescriptionsUtil.java b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/resource/extensions/ResourceDescriptionsUtil.java
index 0e0419ced..c6aa55f1f 100644
--- a/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/resource/extensions/ResourceDescriptionsUtil.java
+++ b/com.avaloq.tools.ddk.xtext/src/com/avaloq/tools/ddk/xtext/resource/extensions/ResourceDescriptionsUtil.java
@@ -21,8 +21,6 @@
 
 import com.avaloq.tools.ddk.xtext.naming.QualifiedNames;
 import com.avaloq.tools.ddk.xtext.resource.extensions.IResourceDescriptions2.ReferenceMatchPolicy;
-import com.google.common.base.Function;
-import com.google.common.base.Predicate;
 import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Iterables;
 import com.google.common.collect.Sets;
@@ -217,17 +215,8 @@ public static Iterable<IReferenceDescription> findReferencesToObjects(final IRes
    * @return An {@link Iterable} of all {@link IReferenceDescription}s of all cross-references that reference the given objects.
    */
   public static Iterable<IReferenceDescription> findReferencesToObjects(final Iterable<IResourceDescription> descriptions, final Set<URI> targetObjects) {
-    return Iterables.concat(Iterables.transform(descriptions, new Function<IResourceDescription, Iterable<IReferenceDescription>>() {
-      @Override
-      public Iterable<IReferenceDescription> apply(final IResourceDescription from) {
-        return Iterables.filter(from.getReferenceDescriptions(), new Predicate<IReferenceDescription>() {
-          @Override
-          public boolean apply(final IReferenceDescription input) {
-            return targetObjects.contains(input.getTargetEObjectUri());
-          }
-        });
-      }
-    }));
+    return Iterables.concat(Iterables.transform(descriptions, from -> Iterables.filter(//
+        from.getReferenceDescriptions(), input -> targetObjects.contains(input.getTargetEObjectUri()))));
   }
 
 }