Use the return value of `addAll()` instead of making separate calls to `containsAll()` and `addAll()`.

RELNOTES=n/a
PiperOrigin-RevId: 375727307

1 files changed, 1 insertions, 2 deletions

diff --git a/service/processor/src/main/java/com/google/auto/service/processor/AutoServiceProcessor.java b/service/processor/src/main/java/com/google/auto/service/processor/AutoServiceProcessor.java
index e70cf6c9..932a0fdf 100644
--- a/service/processor/src/main/java/com/google/auto/service/processor/AutoServiceProcessor.java
+++ b/service/processor/src/main/java/com/google/auto/service/processor/AutoServiceProcessor.java
@@ -186,12 +186,11 @@ public class AutoServiceProcessor extends AbstractProcessor {
         }
 
         Set<String> newServices = new HashSet<>(providers.get(providerInterface));
-        if (allServices.containsAll(newServices)) {
+        if (!allServices.addAll(newServices)) {
           log("No new service entries being added.");
           continue;
         }
 
-        allServices.addAll(newServices);
         log("New service file contents: " + allServices);
         FileObject fileObject = filer.createResource(StandardLocation.CLASS_OUTPUT, "",
             resourceFile);