unused code check

Author: Vincent Potucek

api/maven-api-core/src/main/java/org/apache/maven/api/services/DependencyResolverResult.java

@@ -103,44 +103,4 @@ public interface DependencyResolverResult extends Result<DependencyResolverReque
     @Nonnull
     Map<Dependency, Path> getDependencies();
 
-    /**
-     * Returns the Java module name of the dependency at the given path.
-     * The given dependency should be one of the paths returned by {@link #getDependencies()}.
-     * The module name is extracted from the {@code module-info.class} file if present, otherwise from
-     * the {@code "Automatic-Module-Name"} attribute of the {@code META-INF/MANIFEST.MF} file if present.
-     *
-     * <p>A typical usage is to invoke this method for all dependencies having a
-     * {@link DependencyScope#TEST TEST} or {@link DependencyScope#TEST_ONLY TEST_ONLY}
-     * {@linkplain Dependency#getScope() scope}. An {@code --add-reads} option may need
-     * to be generated for compiling and running the test classes that use such dependencies.</p>
-     *
-     * @param dependency path to the dependency for which to get the module name
-     * @return module name of the dependency at the given path, or empty if the dependency is not modular
-     * @throws IOException if the module information of the specified dependency cannot be read
-     */
-    Optional<String> getModuleName(@Nonnull Path dependency) throws IOException;
-
-    /**
-     * Returns the Java module descriptor of the dependency at the given path.
-     * The given dependency should be one of the paths returned by {@link #getDependencies()}.
-     * The module descriptor is extracted from the {@code module-info.class} file if present.
-     *
-     * <p>{@link #getModuleName(Path)} is preferred when only the module name is desired,
-     * because a name may be present even if the descriptor is absent. This method is for
-     * cases when more information is desired, such as the set of exported packages.</p>
-     *
-     * @param dependency path to the dependency for which to get the module name
-     * @return module name of the dependency at the given path, or empty if the dependency is not modular
-     * @throws IOException if the module information of the specified dependency cannot be read
-     */
-    Optional<ModuleDescriptor> getModuleDescriptor(@Nonnull Path dependency) throws IOException;
-
-    /**
-     * If the module path contains at least one filename-based auto-module, prepares a warning message.
-     * The module path is the collection of dependencies associated with {@link JavaPathType#MODULES}.
-     * It is caller's responsibility to send the message to a logger.
-     *
-     * @return warning message if at least one filename-based auto-module was found
-     */
-    Optional<String> warningForFilenameBasedAutomodules();
 }

impl/maven-impl/src/main/java/org/apache/maven/impl/DefaultDependencyResolverResult.java

@@ -378,18 +378,6 @@ public Map<Dependency, Path> getDependencies() {
         return dependencies;
     }
 
-    @Override
-    public Optional<ModuleDescriptor> getModuleDescriptor(Path dependency) throws IOException {
-        Object value = cache.getModuleInfo(dependency).descriptors.get(dependency);
-        return (value instanceof ModuleDescriptor moduleDescriptor) ? Optional.of(moduleDescriptor) : Optional.empty();
-    }
-
-    @Override
-    public Optional<String> getModuleName(Path dependency) throws IOException {
-        return Optional.ofNullable(
-                name(cache.getModuleInfo(dependency).descriptors.get(dependency)));
-    }
-
     /**
      * Returns the module name for the given value of the {@link PathModularization#descriptors} map.
      */
@@ -403,12 +391,4 @@ private static String name(final Object value) {
         }
     }
 
-    @Override
-    public Optional<String> warningForFilenameBasedAutomodules() {
-        try {
-            return cache.warningForFilenameBasedAutomodules(dispatchedPaths.get(JavaPathType.MODULES));
-        } catch (IOException e) {
-            throw new DependencyResolverException("Cannot read module information.", e);
-        }
-    }
 }

impl/maven-impl/src/main/java/org/apache/maven/impl/PathModularizationCache.java

@@ -167,7 +167,7 @@ Optional<PathType> selectPathType(Set<PathType> types, Predicate<PathType> filte
      * @return warning message if at least one filename-based auto-module was found
      * @throws IOException if an error occurred while reading module information
      */
-    Optional<String> warningForFilenameBasedAutomodules(Collection<Path> modulePaths) throws IOException {
+    Optional<String> _warningForFilenameBasedAutomodules(Collection<Path> modulePaths) throws IOException {
         if (modulePaths == null) {
             return Optional.empty();
         }