Remove BindingReflections.getQName(Rpc)
[mdsal.git] / binding / mdsal-binding-spec-util / src / main / java / org / opendaylight / mdsal / binding / spec / reflect / BindingReflections.java
index 2a0cbed23da3be91ff41ca4f0cb0b21f546abfb5..623385d8658c51906a30874e5fce2b83b639e9d2 100644 (file)
@@ -9,46 +9,25 @@ package org.opendaylight.mdsal.binding.spec.reflect;
 
 import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Preconditions.checkState;
-import static java.util.Objects.requireNonNull;
 
-import com.google.common.annotations.Beta;
+import com.google.common.annotations.VisibleForTesting;
 import com.google.common.cache.CacheBuilder;
 import com.google.common.cache.CacheLoader;
 import com.google.common.cache.LoadingCache;
-import com.google.common.collect.ImmutableSet;
-import com.google.common.collect.ImmutableSet.Builder;
-import com.google.common.util.concurrent.ListenableFuture;
 import java.lang.reflect.Field;
-import java.lang.reflect.Method;
-import java.lang.reflect.Type;
-import java.util.Arrays;
-import java.util.HashMap;
-import java.util.HashSet;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.Map;
+import java.lang.reflect.InvocationTargetException;
 import java.util.Optional;
-import java.util.ServiceLoader;
-import java.util.Set;
 import java.util.concurrent.TimeUnit;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-import org.checkerframework.checker.regex.qual.Regex;
 import org.eclipse.jdt.annotation.NonNull;
-import org.opendaylight.mdsal.binding.spec.naming.BindingMapping;
-import org.opendaylight.yangtools.util.ClassLoaderUtils;
 import org.opendaylight.yangtools.yang.binding.Action;
-import org.opendaylight.yangtools.yang.binding.Augmentable;
 import org.opendaylight.yangtools.yang.binding.Augmentation;
-import org.opendaylight.yangtools.yang.binding.AugmentationHolder;
 import org.opendaylight.yangtools.yang.binding.BaseIdentity;
 import org.opendaylight.yangtools.yang.binding.ChildOf;
 import org.opendaylight.yangtools.yang.binding.DataContainer;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.binding.Notification;
-import org.opendaylight.yangtools.yang.binding.RpcService;
-import org.opendaylight.yangtools.yang.binding.YangModelBindingProvider;
 import org.opendaylight.yangtools.yang.binding.YangModuleInfo;
+import org.opendaylight.yangtools.yang.binding.contract.Naming;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.common.QNameModule;
 import org.opendaylight.yangtools.yang.common.YangConstants;
@@ -56,74 +35,14 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public final class BindingReflections {
-
-    private static final long EXPIRATION_TIME = 60;
-
-    @Regex
-    private static final String ROOT_PACKAGE_PATTERN_STRING =
-            "(org.opendaylight.yang.gen.v1.[a-z0-9_\\.]*\\.(?:rev[0-9][0-9][0-1][0-9][0-3][0-9]|norev))";
-    private static final Pattern ROOT_PACKAGE_PATTERN = Pattern.compile(ROOT_PACKAGE_PATTERN_STRING);
     private static final Logger LOG = LoggerFactory.getLogger(BindingReflections.class);
-
     private static final LoadingCache<Class<?>, Optional<QName>> CLASS_TO_QNAME = CacheBuilder.newBuilder()
             .weakKeys()
-            .expireAfterAccess(EXPIRATION_TIME, TimeUnit.SECONDS)
+            .expireAfterAccess(60, TimeUnit.SECONDS)
             .build(new ClassToQNameLoader());
 
-    private static final LoadingCache<ClassLoader, ImmutableSet<YangModuleInfo>> MODULE_INFO_CACHE =
-            CacheBuilder.newBuilder().weakKeys().weakValues().build(
-                new CacheLoader<ClassLoader, ImmutableSet<YangModuleInfo>>() {
-                    @Override
-                    public ImmutableSet<YangModuleInfo> load(final ClassLoader key) {
-                        return loadModuleInfos(key);
-                    }
-                });
-
     private BindingReflections() {
-        throw new UnsupportedOperationException("Utility class.");
-    }
-
-    /**
-     * Find augmentation target class from concrete Augmentation class. This method uses first generic argument of
-     * implemented {@link Augmentation} interface.
-     *
-     * @param augmentation
-     *            {@link Augmentation} subclass for which we want to determine
-     *            augmentation target.
-     * @return Augmentation target - class which augmentation provides additional extensions.
-     */
-    public static Class<? extends Augmentable<?>> findAugmentationTarget(
-            final Class<? extends Augmentation<?>> augmentation) {
-        final Optional<Class<Augmentable<?>>> opt = ClassLoaderUtils.findFirstGenericArgument(augmentation,
-            Augmentation.class);
-        return opt.orElse(null);
-    }
-
-    /**
-     * Find data hierarchy parent from concrete Data class. This method uses first generic argument of implemented
-     * {@link ChildOf} interface.
-     *
-     * @param childClass
-     *            child class for which we want to find the parent class.
-     * @return Parent class, e.g. class of which the childClass is ChildOf.
-     */
-    public static Class<?> findHierarchicalParent(final Class<? extends ChildOf<?>> childClass) {
-        return ClassLoaderUtils.findFirstGenericArgument(childClass, ChildOf.class).orElse(null);
-    }
-
-    /**
-     * Find data hierarchy parent from concrete Data class. This method is shorthand which gets DataObject class by
-     * invoking {@link DataObject#implementedInterface()} and uses {@link #findHierarchicalParent(Class)}.
-     *
-     * @param child
-     *            Child object for which the parent needs to be located.
-     * @return Parent class, or null if a parent is not found.
-     */
-    public static Class<?> findHierarchicalParent(final DataObject child) {
-        if (child instanceof ChildOf) {
-            return ClassLoaderUtils.findFirstGenericArgument(child.implementedInterface(), ChildOf.class).orElse(null);
-        }
-        return null;
+        // Hidden on purpose
     }
 
     /**
@@ -138,145 +57,45 @@ public final class BindingReflections {
         return CLASS_TO_QNAME.getUnchecked(dataType).orElse(null);
     }
 
-    /**
-     * Checks if method is RPC invocation.
-     *
-     * @param possibleMethod
-     *            Method to check
-     * @return true if method is RPC invocation, false otherwise.
-     */
-    public static boolean isRpcMethod(final Method possibleMethod) {
-        return possibleMethod != null && RpcService.class.isAssignableFrom(possibleMethod.getDeclaringClass())
-                && ListenableFuture.class.isAssignableFrom(possibleMethod.getReturnType())
-                // length <= 2: it seemed to be impossible to get correct RpcMethodInvoker because of
-                // resolveRpcInputClass() check.While RpcMethodInvoker counts with one argument for
-                // non input type and two arguments for input type, resolveRpcInputClass() counting
-                // with zero for non input and one for input type
-                && possibleMethod.getParameterCount() <= 2;
-    }
-
-    /**
-     * Extracts Output class for RPC method.
-     *
-     * @param targetMethod
-     *            method to scan
-     * @return Optional.empty() if result type could not be get, or return type is Void.
-     */
-    @SuppressWarnings("rawtypes")
-    public static Optional<Class<?>> resolveRpcOutputClass(final Method targetMethod) {
-        checkState(isRpcMethod(targetMethod), "Supplied method is not a RPC invocation method");
-        Type futureType = targetMethod.getGenericReturnType();
-        Type rpcResultType = ClassLoaderUtils.getFirstGenericParameter(futureType).orElse(null);
-        Type rpcResultArgument = ClassLoaderUtils.getFirstGenericParameter(rpcResultType).orElse(null);
-        if (rpcResultArgument instanceof Class && !Void.class.equals(rpcResultArgument)) {
-            return Optional.of((Class) rpcResultArgument);
-        }
-        return Optional.empty();
-    }
-
-    /**
-     * Extracts input class for RPC method.
-     *
-     * @param targetMethod
-     *            method to scan
-     * @return Optional.empty() if RPC has no input, RPC input type otherwise.
-     */
-    @SuppressWarnings("rawtypes")
-    public static Optional<Class<? extends DataContainer>> resolveRpcInputClass(final Method targetMethod) {
-        for (Class clazz : targetMethod.getParameterTypes()) {
-            if (DataContainer.class.isAssignableFrom(clazz)) {
-                return Optional.of(clazz);
-            }
-        }
-        return Optional.empty();
-    }
-
-    public static @NonNull QName getQName(final Class<? extends BaseIdentity> bindingClass) {
-        final Optional<QName> qname = CLASS_TO_QNAME.getUnchecked(requireNonNull(bindingClass));
-        checkState(qname.isPresent(), "Failed to resolve QName of %s", bindingClass);
-        return qname.get();
-    }
-
-    /**
-     * Checks if class is child of augmentation.
-     */
-    public static boolean isAugmentationChild(final Class<?> clazz) {
-        // FIXME: Current resolver could be still confused when child node was added by grouping
-        checkArgument(clazz != null);
-
-        @SuppressWarnings({ "rawtypes", "unchecked" })
-        Class<?> parent = findHierarchicalParent((Class) clazz);
-        if (parent == null) {
-            LOG.debug("Did not find a parent for class {}", clazz);
-            return false;
-        }
-
-        String clazzModelPackage = getModelRootPackageName(clazz.getPackage());
-        String parentModelPackage = getModelRootPackageName(parent.getPackage());
-
-        return !clazzModelPackage.equals(parentModelPackage);
-    }
-
-    /**
-     * Returns root package name for supplied package.
-     *
-     * @param pkg
-     *            Package for which find model root package.
-     * @return Package of model root.
-     */
-    public static String getModelRootPackageName(final Package pkg) {
-        return getModelRootPackageName(pkg.getName());
-    }
-
-    /**
-     * Returns root package name for supplied package name.
-     *
-     * @param name
-     *            Package for which find model root package.
-     * @return Package of model root.
-     */
-    public static String getModelRootPackageName(final String name) {
-        checkArgument(name != null, "Package name should not be null.");
-        checkArgument(name.startsWith(BindingMapping.PACKAGE_PREFIX), "Package name not starting with %s, is: %s",
-                BindingMapping.PACKAGE_PREFIX, name);
-        Matcher match = ROOT_PACKAGE_PATTERN.matcher(name);
-        checkArgument(match.find(), "Package name '%s' does not match required pattern '%s'", name,
-                ROOT_PACKAGE_PATTERN_STRING);
-        return match.group(0);
+    public static @NonNull QName getQName(final BaseIdentity identity) {
+        return CLASS_TO_QNAME.getUnchecked(identity.implementedInterface())
+            .orElseThrow(() -> new IllegalStateException("Failed to resolve QName of " + identity));
     }
 
-    @SuppressWarnings("checkstyle:illegalCatch")
     public static QNameModule getQNameModule(final Class<?> clz) {
         if (DataContainer.class.isAssignableFrom(clz) || BaseIdentity.class.isAssignableFrom(clz)
                 || Action.class.isAssignableFrom(clz)) {
             return findQName(clz).getModule();
         }
-        try {
-            return BindingReflections.getModuleInfo(clz).getName().getModule();
-        } catch (Exception e) {
-            throw new IllegalStateException("Unable to get QName of defining model.", e);
-        }
+
+        return getModuleInfo(clz).getName().getModule();
     }
 
     /**
      * Returns instance of {@link YangModuleInfo} of declaring model for specific class.
      *
      * @param cls data object class
-     * @return Instance of {@link YangModuleInfo} associated with model, from
-     *         which this class was derived.
+     * @return Instance of {@link YangModuleInfo} associated with model, from which this class was derived.
      */
-    public static YangModuleInfo getModuleInfo(final Class<?> cls) throws Exception {
-        checkArgument(cls != null);
-        String packageName = getModelRootPackageName(cls.getPackage());
-        final String potentialClassName = getModuleInfoClassName(packageName);
-        return ClassLoaderUtils.callWithClassLoader(cls.getClassLoader(), () -> {
-            Class<?> moduleInfoClass = Thread.currentThread().getContextClassLoader().loadClass(potentialClassName);
-            return (YangModuleInfo) moduleInfoClass.getMethod("getInstance").invoke(null);
-        });
-    }
+    private static @NonNull YangModuleInfo getModuleInfo(final Class<?> cls) {
+        final String potentialClassName = Naming.getModelRootPackageName(cls.getPackage().getName()) + "."
+            + Naming.MODULE_INFO_CLASS_NAME;
+        final Class<?> moduleInfoClass;
+        try {
+            moduleInfoClass = cls.getClassLoader().loadClass(potentialClassName);
+        } catch (ClassNotFoundException e) {
+            throw new IllegalStateException("Failed to load " + potentialClassName, e);
+        }
+
+        final Object infoInstance;
+        try {
+            infoInstance = moduleInfoClass.getMethod("getInstance").invoke(null);
+        } catch (IllegalAccessException | InvocationTargetException | NoSuchMethodException e) {
+            throw new IllegalStateException("Failed to get instance of " + moduleInfoClass, e);
+        }
 
-    public static String getModuleInfoClassName(final String packageName) {
-        return packageName + "." + BindingMapping.MODULE_INFO_CLASS_NAME;
+        checkState(infoInstance instanceof YangModuleInfo, "Unexpected instance %s", infoInstance);
+        return (YangModuleInfo) infoInstance;
     }
 
     /**
@@ -290,105 +109,7 @@ public final class BindingReflections {
         if (DataContainer.class.isAssignableFrom(cls) || Augmentation.class.isAssignableFrom(cls)) {
             return true;
         }
-        return cls.getName().startsWith(BindingMapping.PACKAGE_PREFIX);
-    }
-
-    /**
-     * Checks if supplied method is callback for notifications.
-     *
-     * @param method method to check
-     * @return true if method is notification callback.
-     */
-    public static boolean isNotificationCallback(final Method method) {
-        checkArgument(method != null);
-        if (method.getName().startsWith("on") && method.getParameterCount() == 1) {
-            Class<?> potentialNotification = method.getParameterTypes()[0];
-            if (isNotification(potentialNotification)
-                    && method.getName().equals("on" + potentialNotification.getSimpleName())) {
-                return true;
-            }
-        }
-        return false;
-    }
-
-    /**
-     * Checks is supplied class is a {@link Notification}.
-     *
-     * @param potentialNotification class to examine
-     * @return True if the class represents a Notification.
-     */
-    public static boolean isNotification(final Class<?> potentialNotification) {
-        checkArgument(potentialNotification != null, "potentialNotification must not be null.");
-        return Notification.class.isAssignableFrom(potentialNotification);
-    }
-
-    /**
-     * Loads {@link YangModuleInfo} infos available on current classloader. This method is shorthand for
-     * {@link #loadModuleInfos(ClassLoader)} with {@link Thread#getContextClassLoader()} for current thread.
-     *
-     * @return Set of {@link YangModuleInfo} available for current classloader.
-     */
-    public static ImmutableSet<YangModuleInfo> loadModuleInfos() {
-        return loadModuleInfos(Thread.currentThread().getContextClassLoader());
-    }
-
-    /**
-     * Loads {@link YangModuleInfo} infos available on supplied classloader.
-     *
-     * <p>
-     * {@link YangModuleInfo} are discovered using {@link ServiceLoader} for {@link YangModelBindingProvider}.
-     * {@link YangModelBindingProvider} are simple classes which holds only pointers to actual instance
-     * {@link YangModuleInfo}.
-     *
-     * <p>
-     * When {@link YangModuleInfo} is available, all dependencies are recursively collected into returning set by
-     * collecting results of {@link YangModuleInfo#getImportedModules()}.
-     *
-     * <p>
-     * Consider using {@link #cacheModuleInfos(ClassLoader)} if the classloader is known to be immutable.
-     *
-     * @param loader Classloader for which {@link YangModuleInfo} should be retrieved.
-     * @return Set of {@link YangModuleInfo} available for supplied classloader.
-     */
-    public static ImmutableSet<YangModuleInfo> loadModuleInfos(final ClassLoader loader) {
-        Builder<YangModuleInfo> moduleInfoSet = ImmutableSet.builder();
-        ServiceLoader<YangModelBindingProvider> serviceLoader = ServiceLoader.load(YangModelBindingProvider.class,
-                loader);
-        for (YangModelBindingProvider bindingProvider : serviceLoader) {
-            YangModuleInfo moduleInfo = bindingProvider.getModuleInfo();
-            checkState(moduleInfo != null, "Module Info for %s is not available.", bindingProvider.getClass());
-            collectYangModuleInfo(bindingProvider.getModuleInfo(), moduleInfoSet);
-        }
-        return moduleInfoSet.build();
-    }
-
-    /**
-     * Loads {@link YangModuleInfo} instances available on supplied {@link ClassLoader}, assuming the set of available
-     * information does not change. Subsequent accesses may return cached values.
-     *
-     * <p>
-     * {@link YangModuleInfo} are discovered using {@link ServiceLoader} for {@link YangModelBindingProvider}.
-     * {@link YangModelBindingProvider} are simple classes which holds only pointers to actual instance
-     * {@link YangModuleInfo}.
-     *
-     * <p>
-     * When {@link YangModuleInfo} is available, all dependencies are recursively collected into returning set by
-     * collecting results of {@link YangModuleInfo#getImportedModules()}.
-     *
-     * @param loader Class loader for which {@link YangModuleInfo} should be retrieved.
-     * @return Set of {@link YangModuleInfo} available for supplied classloader.
-     */
-    @Beta
-    public static ImmutableSet<YangModuleInfo> cacheModuleInfos(final ClassLoader loader) {
-        return MODULE_INFO_CACHE.getUnchecked(loader);
-    }
-
-    private static void collectYangModuleInfo(final YangModuleInfo moduleInfo,
-            final Builder<YangModuleInfo> moduleInfoSet) {
-        moduleInfoSet.add(moduleInfo);
-        for (YangModuleInfo dependency : moduleInfo.getImportedModules()) {
-            collectYangModuleInfo(dependency, moduleInfoSet);
-        }
+        return cls.getName().startsWith(Naming.PACKAGE_PREFIX);
     }
 
     /**
@@ -398,103 +119,15 @@ public final class BindingReflections {
      *            Class to be checked
      * @return true if class represents RPC Input or RPC Output class.
      */
-    public static boolean isRpcType(final Class<? extends DataObject> targetType) {
+    @VisibleForTesting
+    static boolean isRpcType(final Class<? extends DataObject> targetType) {
         return DataContainer.class.isAssignableFrom(targetType)
                 && !ChildOf.class.isAssignableFrom(targetType)
                 && !Notification.class.isAssignableFrom(targetType)
                 && (targetType.getName().endsWith("Input") || targetType.getName().endsWith("Output"));
     }
 
-    /**
-     * Scans supplied class and returns an iterable of all data children classes.
-     *
-     * @param type
-     *            YANG Modeled Entity derived from DataContainer
-     * @return Iterable of all data children, which have YANG modeled entity
-     */
-    @SuppressWarnings("unchecked")
-    public static Iterable<Class<? extends DataObject>> getChildrenClasses(final Class<? extends DataContainer> type) {
-        checkArgument(type != null, "Target type must not be null");
-        checkArgument(DataContainer.class.isAssignableFrom(type), "Supplied type must be derived from DataContainer");
-        List<Class<? extends DataObject>> ret = new LinkedList<>();
-        for (Method method : type.getMethods()) {
-            Optional<Class<? extends DataContainer>> entity = getYangModeledReturnType(method,
-                BindingMapping.GETTER_PREFIX);
-            if (entity.isPresent()) {
-                ret.add((Class<? extends DataObject>) entity.get());
-            }
-        }
-        return ret;
-    }
-
-    /**
-     * Scans supplied class and returns an iterable of all data children classes.
-     *
-     * @param type
-     *            YANG Modeled Entity derived from DataContainer
-     * @return Iterable of all data children, which have YANG modeled entity
-     */
-    public static Map<Class<?>, Method> getChildrenClassToMethod(final Class<?> type) {
-        return getChildrenClassToMethod(type, BindingMapping.GETTER_PREFIX);
-    }
-
-    private static Map<Class<?>, Method> getChildrenClassToMethod(final Class<?> type, final String prefix) {
-        checkArgument(type != null, "Target type must not be null");
-        checkArgument(DataContainer.class.isAssignableFrom(type), "Supplied type %s must be derived from DataContainer",
-            type);
-        Map<Class<?>, Method> ret = new HashMap<>();
-        for (Method method : type.getMethods()) {
-            Optional<Class<? extends DataContainer>> entity = getYangModeledReturnType(method, prefix);
-            if (entity.isPresent()) {
-                ret.put(entity.get(), method);
-            }
-        }
-        return ret;
-    }
-
-    @Beta
-    public static Map<Class<?>, Method> getChildrenClassToNonnullMethod(final Class<?> type) {
-        return getChildrenClassToMethod(type, BindingMapping.NONNULL_PREFIX);
-    }
-
-    @SuppressWarnings("checkstyle:illegalCatch")
-    private static Optional<Class<? extends DataContainer>> getYangModeledReturnType(final Method method,
-            final String prefix) {
-        final String methodName = method.getName();
-        if ("getClass".equals(methodName) || !methodName.startsWith(prefix) || method.getParameterCount() > 0) {
-            return Optional.empty();
-        }
-
-        final Class<?> returnType = method.getReturnType();
-        if (DataContainer.class.isAssignableFrom(returnType)) {
-            return optionalDataContainer(returnType);
-        } else if (List.class.isAssignableFrom(returnType)) {
-            try {
-                return ClassLoaderUtils.callWithClassLoader(method.getDeclaringClass().getClassLoader(), () -> {
-                    return ClassLoaderUtils.getFirstGenericParameter(method.getGenericReturnType()).flatMap(
-                        result -> result instanceof Class ? optionalCast((Class<?>) result) : Optional.empty());
-                });
-            } catch (Exception e) {
-                /*
-                 * It is safe to log this this exception on debug, since this
-                 * method should not fail. Only failures are possible if the
-                 * runtime / backing.
-                 */
-                LOG.debug("Unable to find YANG modeled return type for {}", method, e);
-            }
-        }
-        return Optional.empty();
-    }
-
-    private static Optional<Class<? extends DataContainer>> optionalCast(final Class<?> type) {
-        return DataContainer.class.isAssignableFrom(type) ? optionalDataContainer(type) : Optional.empty();
-    }
-
-    private static Optional<Class<? extends DataContainer>> optionalDataContainer(final Class<?> type) {
-        return Optional.of(type.asSubclass(DataContainer.class));
-    }
-
-    private static class ClassToQNameLoader extends CacheLoader<Class<?>, Optional<QName>> {
+    private static final class ClassToQNameLoader extends CacheLoader<Class<?>, Optional<QName>> {
 
         @Override
         public Optional<QName> load(@SuppressWarnings("NullableProblems") final Class<?> key) throws Exception {
@@ -503,23 +136,23 @@ public final class BindingReflections {
 
         /**
          * Tries to resolve QName for supplied class. Looks up for static field with name from constant
-         * {@link BindingMapping#QNAME_STATIC_FIELD_NAME} and returns value if present. If field is not present uses
+         * {@link Naming#QNAME_STATIC_FIELD_NAME} and returns value if present. If field is not present uses
          * {@link #computeQName(Class)} to compute QName for missing types.
          */
         private static Optional<QName> resolveQNameNoCache(final Class<?> key) {
             try {
                 final Field field;
                 try {
-                    field = key.getField(BindingMapping.QNAME_STATIC_FIELD_NAME);
+                    field = key.getField(Naming.QNAME_STATIC_FIELD_NAME);
                 } catch (NoSuchFieldException e) {
                     LOG.debug("{} does not have a {} field, falling back to computation", key,
-                        BindingMapping.QNAME_STATIC_FIELD_NAME, e);
+                        Naming.QNAME_STATIC_FIELD_NAME, e);
                     return Optional.of(computeQName(key));
                 }
 
                 final Object obj = field.get(null);
-                if (obj instanceof QName) {
-                    return Optional.of((QName) obj);
+                if (obj instanceof QName qname) {
+                    return Optional.of(qname);
                 }
             } catch (SecurityException | IllegalArgumentException | IllegalAccessException e) {
                 /*
@@ -551,23 +184,16 @@ public final class BindingReflections {
          * @throws IllegalArgumentException If supplied class was not derived from YANG model.
          */
         // FIXME: Extend this algorithm to also provide QName for YANG modeled simple types.
-        @SuppressWarnings({ "rawtypes", "unchecked", "checkstyle:illegalCatch" })
+        @SuppressWarnings({ "rawtypes", "unchecked" })
         private static QName computeQName(final Class key) {
             checkArgument(isBindingClass(key), "Supplied class %s is not derived from YANG.", key);
 
-            YangModuleInfo moduleInfo;
-            try {
-                moduleInfo = getModuleInfo(key);
-            } catch (Exception e) {
-                throw new IllegalStateException("Unable to get QName for " + key + ". YangModuleInfo was not found.",
-                    e);
-            }
-            final QName module = moduleInfo.getName();
+            final QName module = getModuleInfo(key).getName();
             if (Augmentation.class.isAssignableFrom(key)) {
                 return module;
             } else if (isRpcType(key)) {
                 final String className = key.getSimpleName();
-                if (className.endsWith(BindingMapping.RPC_OUTPUT_SUFFIX)) {
+                if (className.endsWith(Naming.RPC_OUTPUT_SUFFIX)) {
                     return YangConstants.operationOutputQName(module.getModule()).intern();
                 }
 
@@ -580,75 +206,4 @@ public final class BindingReflections {
             return module;
         }
     }
-
-    /**
-     * Extracts augmentation from Binding DTO field using reflection.
-     *
-     * @param input
-     *            Instance of DataObject which is augmentable and may contain
-     *            augmentation
-     * @return Map of augmentations if read was successful, otherwise empty map.
-     */
-    public static Map<Class<? extends Augmentation<?>>, Augmentation<?>> getAugmentations(final Augmentable<?> input) {
-        if (input instanceof AugmentationHolder) {
-            return ((AugmentationHolder) input).augmentations();
-        }
-        return AugmentationFieldGetter.getGetter(input.getClass()).getAugmentations(input);
-    }
-
-    /**
-     * Determines if two augmentation classes or case classes represents same
-     * data.
-     *
-     * <p>
-     * Two augmentations or cases could be substituted only if and if:
-     * <ul>
-     * <li>Both implements same interfaces</li>
-     * <li>Both have same children</li>
-     * <li>If augmentations: Both have same augmentation target class. Target
-     * class was generated for data node in grouping.</li>
-     * <li>If cases: Both are from same choice. Choice class was generated for
-     * data node in grouping.</li>
-     * </ul>
-     *
-     * <p>
-     * <b>Explanation:</b> Binding Specification reuses classes generated for
-     * groupings as part of normal data tree, this classes from grouping could
-     * be used at various locations and user may not be aware of it and may use
-     * incorrect case or augmentation in particular subtree (via copy
-     * constructors, etc).
-     *
-     * @param potential
-     *            Class which is potential substitution
-     * @param target
-     *            Class which should be used at particular subtree
-     * @return true if and only if classes represents same data.
-     */
-    @SuppressWarnings({ "rawtypes", "unchecked" })
-    public static boolean isSubstitutionFor(final Class potential, final Class target) {
-        Set<Class> subImplemented = new HashSet<>(Arrays.asList(potential.getInterfaces()));
-        Set<Class> targetImplemented = new HashSet<>(Arrays.asList(target.getInterfaces()));
-        if (!subImplemented.equals(targetImplemented)) {
-            return false;
-        }
-        if (Augmentation.class.isAssignableFrom(potential)
-                && !BindingReflections.findAugmentationTarget(potential).equals(
-                        BindingReflections.findAugmentationTarget(target))) {
-            return false;
-        }
-        for (Method potentialMethod : potential.getMethods()) {
-            try {
-                Method targetMethod = target.getMethod(potentialMethod.getName(), potentialMethod.getParameterTypes());
-                if (!potentialMethod.getReturnType().equals(targetMethod.getReturnType())) {
-                    return false;
-                }
-            } catch (NoSuchMethodException e) {
-                // Counterpart method is missing, so classes could not be substituted.
-                return false;
-            } catch (SecurityException e) {
-                throw new IllegalStateException("Could not compare methods", e);
-            }
-        }
-        return true;
-    }
 }