Hi Stuart,
Filed an issue for the cleanup:
https://bugs.openjdk.java.net/browse/JDK-8198989
Naoto
On 3/2/18 5:50 PM, Stuart Marks wrote:
Looks good.
I note that other codepoint-consuming methods, such as
Character.UnicodeBlock.of(cp)
Character.UnicodeScript.of(cp)
Character.toChars(cp, char[], int)
Character.toChars(cp)
Character.getName(cp)
all throw IAE with no message. It would be nice to add messages to them.
It would be even nicer to print out the offending value, possibly even
in hex. Indeed, there are several other places in Character.java where
exceptions are thrown that lack diagnostic information. Maybe as part of
a separate cleanup pass?
s'marks
On 3/2/18 3:37 PM, naoto.s...@oracle.com wrote:
Thanks for comments, Martin, Roger. Updated the fix as follows:
http://cr.openjdk.java.net/~naoto/4993841/webrev.04/
Naoto
On 3/1/18 6:47 PM, naoto.s...@oracle.com wrote:
Hi,
Please review the fix to the following issue:
https://bugs.openjdk.java.net/browse/JDK-4993841
The proposed changeset is located at:
http://cr.openjdk.java.net/~naoto/4993841/webrev.03/
This stems from the recent discussion regarding String.repeat().[1]
The corresponding CSR has already been approved.
Naoto
--
[1]
http://mail.openjdk.java.net/pipermail/core-libs-dev/2018-February/051568.html