cowwoc commented on a change in pull request #14: Exclude non-exported packages
when Java Modules are present.
URL:
https://github.com/apache/maven-javadoc-plugin/pull/14#discussion_r243923489
##########
File path:
src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
##########
@@ -4342,6 +4351,102 @@ private void copyAdditionalJavadocResources( File
anOutputDirectory )
return getPackageNamesOrFilesWithUnnamedPackages( sourcePaths, files,
true );
}
+ /**
+ * @param allSourcePaths not null, containing absolute and relative
paths
+ * @return a list of exported package names for files in allSourcePaths
+ * @throws MavenReportException if any
+ * @see #getFiles
+ * @see #getSourcePaths()
+ */
+ private List<String> getPackageNamesRespectingJavaModules( Map<String,
Collection<String>> allSourcePaths )
+ throws MavenReportException
+ {
+ List<String> returnList = new ArrayList<>();
+
+ if ( !StringUtils.isEmpty( sourcepath ) )
+ {
+ return returnList;
+ }
+ LocationManager locationManager = new LocationManager();
Review comment:
This method only gets invoked once per goal-call. Anyway, feel free to
refactor this code to your liking. I don't mind :)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
