RFR 2: JDK-8005263: Logging APIs takes Supplier for message (original) (raw)

Peter Levart peter.levart at gmail.com
Sat Dec 22 11:13:55 UTC 2012

Hi Henry,

Plain and simple.

I just noticed a few spellings:

line 670, 696, 863: "Thus is it" -> "Thus it is" - this one has already been written wrong in the original source, so it just multiplied by copy-paste line 688, 743, 854: "Log a in-time" -> "Log an in-time"

Regards, Peter

On 12/22/2012 05:50 AM, Henry Jen wrote:

Hi,

Update patch with review feedback, - JavaDoc update for benefit and gotcha. - logEx/logpEx is not log/logp with Supplier as last argument. As a matter of fact, all API with Supplier takes it as last argument. - No more doLog(Level, Supplier, Block) helper for performance concerns. Specdiff and webrev can be found at following, http://cr.openjdk.java.net/~henryjen/ccc/8005263.1/specdiff/diff.html http://cr.openjdk.java.net/~henryjen/ccc/8005263.1/webrev/ Cheers, Henry

More information about the core-libs-dev mailing list