Thanks!
On 05.03.2015 20:48, Martin Buchholz wrote:
Looks good to me!
On Thu, Mar 5, 2015 at 7:12 AM, Ivan Gerasimov
<ivan.gerasi...@oracle.com <mailto:ivan.gerasi...@oracle.com>> wrote:
Hello!
It was noticed, that the javadoc for CharsetDecoder#replaceWith is
missing the words about constraints on the length of the new
replacement string [1].
Another small glitch is that the common part (of both encoder and
decoder) of the 'newReplacement' param description is prepended to
each specific description, so they read like that: "The
replacement value The new replacement; ..."
Would you please help review the fix?
BUGURL: https://bugs.openjdk.java.net/browse/JDK-8073692
WEBREV: http://cr.openjdk.java.net/~igerasim/8073692/0/webrev/
<http://cr.openjdk.java.net/%7Eigerasim/8073692/0/webrev/>
Sincerely yours,
Ivan
[1]
http://docs.oracle.com/javase/8/docs/api/java/nio/charset/CharsetDecoder.html#replaceWith-java.lang.String-
