On 10/11/18 4:54 AM, Daniel Fuchs wrote:
Hi Mandy,

On 10/10/2018 17:56, Mandy Chung wrote:
The javadoc of SimpleFormatter::format(LogRecord) method does
describe the format argument is a java.util.Formatter format string.
Also the class description also links to this method.  So I think these
two broken links can simply be removed and the spec about the formatting
is in the SimpleFormatter::format method javadoc.  What do you think?


I agree, this is probably the better solution:

http://cr.openjdk.java.net/~dfuchs/webrev_8211960/webrev.01/index.html

Looks good.

+ * @see #format(LogRecord)

This might not be needed since it's linked from

{@linkplain #format(LogRecord) format} the log messages.

No need for a new webrev.

Thanks
Mandy

Reply via email to