RFR: JDK-8168836 Minor clean up on warning/error messages
Mandy Chung
mandy.chung at oracle.com
Mon Nov 14 23:20:33 UTC 2016
> On Nov 14, 2016, at 1:22 PM, Patrick Reinhart <patrick at reini.net> wrote:
>
> Ups, previous mail contained wrong URL. Here’s the correct one:
>
> http://cr.openjdk.java.net/~reinhapa/reviews/8168836/webrev.00
879 log.println(getMessage("err.missing.dependences"));
This should call reportError method that will prepend with “error.prefix”. So no need to have “ERROR:” in “err.missing.dependences” message.
Formatting Nit: line 541-542 and line 753-754 align to the first parameter of the method.
Otherwise looks good. I can sponsor it.
Mandy
More information about the core-libs-dev
mailing list