RFR: JDK-8185359: Unnecessary <p> in module doc comment

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

RFR: JDK-8185359: Unnecessary <p> in module doc comment

Jonathan Gibbons
Continuing the ongoing docs cleanup, please review this fix to remove a
superfluous <p>
in the docs for the java.scripting module.

No webrev; just one line deleted, as shown here:

$ hg diff -R jdk
diff -r d93f2fd542b7 src/java.scripting/share/classes/module-info.java
--- a/src/java.scripting/share/classes/module-info.java Tue Jul 25
18:36:19 2017 -0700
+++ b/src/java.scripting/share/classes/module-info.java Wed Jul 26
11:49:07 2017 -0700
@@ -31,7 +31,6 @@
   * that supports executing JavaScript and other languages if its
corresponding
   * script engine is installed.
   *
- * <p>
   * <dl style="font-family:'DejaVu Sans', Arial, Helvetica, sans serif">
   * <dt class="simpleTagLabel">Tool Guides:
   * <dd> {@extLink jrunscript_tool_reference jrunscript}</dd>

JBS: https://bugs.openjdk.java.net/browse/JDK-8185359

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

Re: RFR: JDK-8185359: Unnecessary <p> in module doc comment

Lance Andersen
+1

> On Jul 26, 2017, at 2:57 PM, Jonathan Gibbons <[hidden email]> wrote:
>
> Continuing the ongoing docs cleanup, please review this fix to remove a superfluous <p>
> in the docs for the java.scripting module.
>
> No webrev; just one line deleted, as shown here:
>
> $ hg diff -R jdk
> diff -r d93f2fd542b7 src/java.scripting/share/classes/module-info.java
> --- a/src/java.scripting/share/classes/module-info.java Tue Jul 25 18:36:19 2017 -0700
> +++ b/src/java.scripting/share/classes/module-info.java Wed Jul 26 11:49:07 2017 -0700
> @@ -31,7 +31,6 @@
>  * that supports executing JavaScript and other languages if its corresponding
>  * script engine is installed.
>  *
> - * <p>
>  * <dl style="font-family:'DejaVu Sans', Arial, Helvetica, sans serif">
>  * <dt class="simpleTagLabel">Tool Guides:
>  * <dd> {@extLink jrunscript_tool_reference jrunscript}</dd>
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8185359
>
> -- Jon

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
[hidden email] <mailto:[hidden email]>



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

Re: RFR: JDK-8185359: Unnecessary <p> in module doc comment

Mandy Chung
In reply to this post by Jonathan Gibbons
+1

Mandy

> On Jul 26, 2017, at 11:57 AM, Jonathan Gibbons <[hidden email]> wrote:
>
> Continuing the ongoing docs cleanup, please review this fix to remove a superfluous <p>
> in the docs for the java.scripting module.
>
> No webrev; just one line deleted, as shown here:
>
> $ hg diff -R jdk
> diff -r d93f2fd542b7 src/java.scripting/share/classes/module-info.java
> --- a/src/java.scripting/share/classes/module-info.java Tue Jul 25 18:36:19 2017 -0700
> +++ b/src/java.scripting/share/classes/module-info.java Wed Jul 26 11:49:07 2017 -0700
> @@ -31,7 +31,6 @@
>  * that supports executing JavaScript and other languages if its corresponding
>  * script engine is installed.
>  *
> - * <p>
>  * <dl style="font-family:'DejaVu Sans', Arial, Helvetica, sans serif">
>  * <dt class="simpleTagLabel">Tool Guides:
>  * <dd> {@extLink jrunscript_tool_reference jrunscript}</dd>
>
> JBS: https://bugs.openjdk.java.net/browse/JDK-8185359
>
> -- Jon

Loading...