/*
 * Licensed to the Apache Software Foundation (ASF) under one or more
 * contributor license agreements. See the NOTICE file distributed with
 * this work for additional information regarding copyright ownership.
 * The ASF licenses this file to You 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.apache.logging.log4j.core.config.plugins.visitors;

import java.lang.annotation.Annotation;

import org.apache.logging.log4j.Logger;
import org.apache.logging.log4j.core.config.plugins.PluginVisitorStrategy;
import org.apache.logging.log4j.status.StatusLogger;

/**
 * Utility class to locate an appropriate PluginVisitor implementation for an annotation.
 */
public final class PluginVisitors {

    private static final Logger LOGGER = StatusLogger.getLogger();

    private PluginVisitors() {
    }

    /**
     * Creates a PluginVisitor instance for the given annotation class using metadata provided by the annotation's
     * {@link PluginVisitorStrategy} annotation. This instance must be further populated with
     * data to be useful. Such data is passed through both the setters and the visit method.
     *
     * @param annotation the Plugin annotation class to find a PluginVisitor for.
     * @return a PluginVisitor instance if one could be created, or {@code null} otherwise.
     */
    public static PluginVisitor<? extends Annotation> findVisitor(final Class<? extends Annotation> annotation) {
        final Class<? extends PluginVisitor<? extends Annotation>> visitorClass = findVisitorStrategy(annotation);
        if (visitorClass == null) {
            LOGGER.debug("No PluginVisitorStrategy found on annotation [{}]. Ignoring.", annotation);
            return null;
        }
        try {
            return visitorClass.newInstance();
        } catch (final Exception e) {
            LOGGER.error("Error loading PluginVisitor [{}] for annotation [{}].", visitorClass, annotation, e);
            return null;
        }
    }

    private static Class<? extends PluginVisitor<? extends Annotation>> findVisitorStrategy(
        final Class<? extends Annotation> annotation) {
        final PluginVisitorStrategy strategy = annotation.getAnnotation(PluginVisitorStrategy.class);
        if (strategy != null) {
            return strategy.value();
        }
        final Annotation[] annotations = annotation.getDeclaredAnnotations();
        for (final Annotation a : annotations) {
            final PluginVisitorStrategy fallbackStrategy = a.annotationType().getAnnotation(
                PluginVisitorStrategy.class);
            if (fallbackStrategy != null) {
                return fallbackStrategy.value();
            }
        }
        return null;
    }
}

@@ -1941,80 +1941,72 @@
 nal 
-Class%3C? extends PluginVisitor%3C? extends Annotation%3E%3E visitorClass = f
+PluginVisitorStrategy strategy = annotation.getAnnotation(Plug
 in
-d
 Visi
@@ -2020,19 +2020,14 @@
 tegy
-(annotation
+.class
 );%0A 
@@ -2037,28 +2037,24 @@
     if (
-visitorClass
+strategy
  == null
@@ -2227,28 +2227,32 @@
  return 
-visitorClass
+strategy.value()
 .newInst
@@ -2383,813 +2383,76 @@
 .%22, 
-visitorClass, annotation, e);%0A            return null;%0A        %7D%0A    %7D%0A%0A    private static Class%3C? extends PluginVisitor%3C? extends Annotation%3E%3E findVisitorStrategy(%0A        final Class%3C? extends Annotation%3E annotation) %7B%0A        final PluginVisitorStrategy strategy = annotation.getAnnotation(PluginVisitorStrategy.class);%0A        if (strategy != null) %7B%0A            return strategy.value();%0A        %7D%0A        final Annotation%5B%5D annotations = annotation.getDeclaredAnnotations();%0A        for (final Annotation a : annotations) %7B%0A            final PluginVisitorStrategy fallbackStrategy = a.annotationType().getAnnotation(%0A                PluginVisitorStrategy.class);%0A            if (fallbackStrategy != null) %7B%0A                return fallbackStrategy.value();%0A            %7D%0A        %7D%0A        return null;
+strategy.value(), annotation, e);%0A            return null;%0A        %7D
 %0A   

Revert unnecessary annotation graph walking.