+1
Naoto
On 5/21/18 10:00 AM, Ivan Gerasimov wrote:
Looks good!
With kind regards,
Ivan
On 5/21/18 9:57 AM, Brian Burkhalter wrote:
https://bugs.openjdk.java.net/browse/JDK-8203487
--- a/src/java.base/share/classes/sun/text/normalizer/ICUBinary.java
+++ b/src/java.base/share/classes/sun/text/normalizer/ICUBinary.java
@@ -255,7 +255,7 @@
* Error messages
*/
private static final String MAGIC_NUMBER_AUTHENTICATION_FAILED_ =
- "ICUBinary data file error: Magin number
authentication failed";
+ "ICUBinary data file error: Magic number
authentication failed";
private static final String HEADER_AUTHENTICATION_FAILED_ =
"ICUBinary data file error: Header authentication failed";
}
Thanks,
Brian
On May 21, 2018, at 9:50 AM, Brian Burkhalter
<[email protected]> wrote:
Hi Adam,
I’ll take it.
Thanks,
Brian
On May 21, 2018, at 9:01 AM, Adam Farley8 <[email protected]>
wrote:
There's a typo here on line 258, where is says "Magin number" but means
"Magic number".
http://hg.openjdk.java.net/jdk/jdk/file/ec881a19d294/src/java.base/share/classes/sun/text/normalizer/ICUBinary.java
Can a committer review and correct the typo?