Hi Joe,

Looks good to me. You might also change the followings if you want:

src/jdk.jdi/share/classes/com/sun/jdi/request/AccessWatchpointRequest.java
------------------------------------------------------------------------

@@ -29,12 +29,11 @@
/**
  * Request for notification when the contents of a field are accessed
  * in the target VM.
  * This event will be triggered when the specified field is accessed
- * by Java<SUP><FONT SIZE="-2">TM</FONT></SUP> programming
- * language code or by a
+ * by Java&trade; programming language code or by a
  * Java Native Interface (JNI) get function*(<code>Get&lt;Type&gt;Field,
  * GetStatic&lt;Type&gt;Field</code>).*


 ({@code Get<Type>Field, GetStatic<Type>Field}).


src/jdk.jdi/share/classes/com/sun/jdi/request/ModificationWatchpointRequest.java
------------------------------------------------------------------------

@@ -28,11 +28,11 @@
 import com.sun.jdi.*;
/**
  * Request for notification when a field is set.
  * This event will be triggered when a value is assigned to the specified
- * field with a Java<SUP><FONT SIZE="-2">TM</FONT></SUP> programming
+ * field with a Java&trade; programming
  * language statement (assignment, increment, etc) or by a
  * Java Native Interface (JNI) set function*(<code>Set&lt;Type&gt;Field,
  * SetStatic&lt;Type&gt;Field</code>).*

({@code Set<Type>Field, SetStatic<Type>Field}).


-Joe

On 5/19/2015 12:41 PM, joe darcy wrote:
Hello,

Please review these changes for

     JDK-8075284: fix up miscellaneous TM constructions
    http://cr.openjdk.java.net/~darcy/8075284.0/

which replace convoluted HTML markup for expressing a trademark sign with the HTML &trade; --- build and docs build pass with the changes

Thanks,

-Joe


Reply via email to