/*
 * Copyright 2018 the original author or authors.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

package org.gradle.api.internal.changedetection.rules;

import org.gradle.api.NonNullApi;
import org.gradle.api.Task;
import org.gradle.api.internal.changedetection.state.TaskExecution;
import org.gradle.util.ChangeListener;
import org.gradle.util.DiffUtil;

import java.util.List;
import java.util.Map;

@NonNullApi
public abstract class AbstractPropertyNameTaskStateChanges extends SimpleTaskStateChanges {

    private final TaskExecution previous;
    private final TaskExecution current;
    private final String title;
    private final Task task;

    protected AbstractPropertyNameTaskStateChanges(TaskExecution previous, TaskExecution current, String title, Task task) {
        this.previous = previous;
        this.current = current;
        this.title = title;
        this.task = task;
    }

    protected abstract Map<String, ?> getProperties(TaskExecution execution);

    @Override
    protected void addAllChanges(final List<TaskStateChange> changes) {
        DiffUtil.diff(getProperties(current).keySet(), getProperties(previous).keySet(), new ChangeListener<String>() {
            @Override
            public void added(String element) {
                changes.add(new DescriptiveChange("%s property '%s' has been added for %s", title, element, task));
            }

            @Override
            public void removed(String element) {
                changes.add(new DescriptiveChange("%s property '%s' has been removed for %s", title, element, task));
            }

            @Override
            public void changed(String element) {
                // Won't happen
                throw new AssertionError();
            }
        });
    }
}

@@ -799,80 +799,8 @@
 ion;
-%0Aimport org.gradle.util.ChangeListener;%0Aimport org.gradle.util.DiffUtil;
 %0A%0Aim
@@ -841,16 +841,38 @@
 til.Map;
+%0Aimport java.util.Set;
 %0A%0A@NonNu
@@ -1544,22 +1544,35 @@
     
-DiffUtil.diff(
+Set%3CString%3E currentNames = 
 getP
@@ -1602,141 +1602,169 @@
 et()
-, getProperties(previous).keySet(), new ChangeListener%3CString%3E() %7B%0A            @Override%0A            public void added(String element
+;%0A        Set%3CString%3E previousNames = getProperties(previous).keySet();%0A%0A        for (String name : currentNames) %7B%0A            if (!previousNames.contains(name)
 ) %7B%0A
@@ -1854,39 +1854,36 @@
 for %25s%22, title, 
-element
+name
 , task));%0A      
@@ -1886,37 +1886,42 @@
           %7D%0A
-%0A
+    
     
+%7D%0A
         
 @Override%0A  
@@ -1912,64 +1912,88 @@
     
-@Override%0A            public void removed(String element
+for (String name : previousNames) %7B%0A            if (!currentNames.contains(name)
 ) %7B%0A
@@ -2097,185 +2097,19 @@
 le, 
-element, task));%0A            %7D%0A%0A            @Override%0A            public void changed(String element) %7B%0A                // Won't happen%0A                throw new AssertionError(
+name, task)
 );%0A 
@@ -2130,18 +2130,16 @@
        %7D
-);
 %0A    %7D%0A%7D

Improve performance of property change calculation