Re: RFR: XXS,doc JDK-8225496: typo in fix for JDK-8224257

2019-06-07 Thread Mandy Chung

+1.  Missed this in the review.

Mandy

On 6/7/19 4:36 PM, Jonathan Gibbons wrote:
The fix for  JDK-8224257 had a transposition typo, putting a ';' 
outside a quoted string instead of inside it.


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

-- Jon


$ hg diff -R open
diff -r fd61ef6c4091 
make/jdk/src/classes/build/tools/fixuppandoc/Main.java
--- a/make/jdk/src/classes/build/tools/fixuppandoc/Main.java Fri Jun 
07 14:32:48 2019 -0700
+++ b/make/jdk/src/classes/build/tools/fixuppandoc/Main.java Fri Jun 
07 16:31:17 2019 -0700

@@ -558,7 +558,7 @@
 out.write(style);
 out.write(m.group("after"));
 } else {
-    out.write(" style=\"font-weight: normal; 
text-align:left\"; ");
+    out.write(" style=\"font-weight: normal; 
text-align:left;\" ");

 out.write(attrs);
 }
 out.write(" scope=\"row\"");





RFR: XXS,doc JDK-8225496: typo in fix for JDK-8224257

2019-06-07 Thread Jonathan Gibbons
The fix for  JDK-8224257 had a transposition typo, putting a ';' outside 
a quoted string instead of inside it.


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

-- Jon


$ hg diff -R open
diff -r fd61ef6c4091 make/jdk/src/classes/build/tools/fixuppandoc/Main.java
--- a/make/jdk/src/classes/build/tools/fixuppandoc/Main.java    Fri Jun 
07 14:32:48 2019 -0700
+++ b/make/jdk/src/classes/build/tools/fixuppandoc/Main.java    Fri Jun 
07 16:31:17 2019 -0700

@@ -558,7 +558,7 @@
 out.write(style);
 out.write(m.group("after"));
 } else {
-    out.write(" style=\"font-weight: normal; 
text-align:left\"; ");
+    out.write(" style=\"font-weight: normal; 
text-align:left;\" ");

 out.write(attrs);
 }
 out.write(" scope=\"row\"");