Sorry, just a reminder.
On 03.06.2014 20:46, alexander stepanov wrote:
Hello Sergey,
Updated; please see
http://cr.openjdk.java.net/~avstepan/8043967/webrev.01/
Thanks,
Alexander
On 03.06.2014 17:56, Sergey Bylokhov wrote:
Hi, Alexander.
A few notes:
- Use two spaces after @param tag only. in other cases use one space.
- Do no align names of the @params tag like this:
390 * position will not be replaced).
391 * @param str the non-{@code null} text to use as
392 * the replacement
393 * @param start the start position
394 * @param end the end position
395 * @deprecated As of JDK version 1.1,
Change them to:
390 * position will not be replaced).
391 * @param str the non-{@code null} text to use as
392 * the replacement
393 * @param start the start position
394 * @param end the end position
395 * @deprecated As of JDK version 1.1,
- Add empty line after method description before @param tag.
- Description of the method should ends by dot.
On 6/3/14 4:48 PM, alexander stepanov wrote:
Hello,
Could you please review the fix for the following bug:
https://bugs.openjdk.java.net/browse/JDK-8043967
Webrev:
http://cr.openjdk.java.net/~avstepan/8043967
Just a cleanup of javadoc to avoid doclint warnings.
Thanks,
Alexander
--
Best regards, Sergey.
