JDK 9 RFR Fix bad javadoc link in javax.tools.JavaFileManager

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

JDK 9 RFR Fix bad javadoc link in javax.tools.JavaFileManager

joe darcy
Hello,

Please review the patch below to fix a bad javadoc link in
JavaFileManager, listLocationsForModule*s* vs listLocationsForModule.

To avoid these problems in the future, I just sent out for review a
build change to enable the doclint reference check for the java.compiler
module.

Thanks,

-Joe

diff -r b87052a3056f
src/java.compiler/share/classes/javax/tools/JavaFileManager.java
--- a/src/java.compiler/share/classes/javax/tools/JavaFileManager.java
Mon Feb 13 11:51:51 2017 -0800
+++ b/src/java.compiler/share/classes/javax/tools/JavaFileManager.java
Mon Feb 13 12:04:58 2017 -0800
@@ -130,7 +130,7 @@
       * It is not possible to directly list the classes in a
module-oriented
       * location. Instead, you can get a package-oriented location for
any specific module
       * using methods like {@link JavaFileManager#getLocationForModule} or
-     * {@link JavaFileManager#listLocationsForModule}.
+     * {@link JavaFileManager#listLocationsForModules}.
       */
      interface Location {
          /**

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: JDK 9 RFR Fix bad javadoc link in javax.tools.JavaFileManager

Jonathan Gibbons
Looks OK to me.  Thanks.

-- Jon

On 02/13/2017 12:06 PM, joe darcy wrote:

> Hello,
>
> Please review the patch below to fix a bad javadoc link in
> JavaFileManager, listLocationsForModule*s* vs listLocationsForModule.
>
> To avoid these problems in the future, I just sent out for review a
> build change to enable the doclint reference check for the
> java.compiler module.
>
> Thanks,
>
> -Joe
>
> diff -r b87052a3056f
> src/java.compiler/share/classes/javax/tools/JavaFileManager.java
> --- a/src/java.compiler/share/classes/javax/tools/JavaFileManager.java
> Mon Feb 13 11:51:51 2017 -0800
> +++ b/src/java.compiler/share/classes/javax/tools/JavaFileManager.java
> Mon Feb 13 12:04:58 2017 -0800
> @@ -130,7 +130,7 @@
>       * It is not possible to directly list the classes in a
> module-oriented
>       * location. Instead, you can get a package-oriented location for
> any specific module
>       * using methods like {@link
> JavaFileManager#getLocationForModule} or
> -     * {@link JavaFileManager#listLocationsForModule}.
> +     * {@link JavaFileManager#listLocationsForModules}.
>       */
>      interface Location {
>          /**
>

Loading...