The fix looks good for me.

  Thanks,
  Alexandr.

On 5/16/2014 1:21 PM, alexander stepanov wrote:
Hello Alexander,

Removed; please find the the updated webrev here:
http://cr.openjdk.java.net/~yan/8042120/webrev.01/

Thanks,
Alexander

On 15.05.2014 19:10, Alexander Scherbatiy wrote:


src/share/classes/javax/swing/text/html/parser/Parser.java
+     * Finally delegates the call to an error with the message
+     * (err + " " + arg1 + " " + arg2 + " " + arg3).

Could you remove this Comment?

The method implementation can be changed and it will require additional CCC request for the javadoc change.

Thanks,
Alexandr.

On 4/29/2014 6:32 PM, alexander stepanov wrote:
Hello,

Could you please review the fix for the following bug:
https://bugs.openjdk.java.net/browse/JDK-8042120

Webrev corresponding:
http://cr.openjdk.java.net/~yan/8042120/webrev.00/

Just a minor cleanup of javadoc to avoid doclint warnings.

Thanks,
Alexander



Reply via email to