OK by me.

We might at some point want to do additional cleanup of the names (the current names are archaic and do not accurately reflect their intent) and we might want to redice the overloads in jdk.javadoc, but for the issue being addressed, the proposed change is good.

-- Jon

On 05/16/2016 04:47 PM, Claes Redestad wrote:
Hi,

please review this patch to avoid using exceptional control flow in .../doclets/internal/toolkit/Configuration.java

Webrev: http://cr.openjdk.java.net/~redestad/8157102/webrev.01/
Bug: https://bugs.openjdk.java.net/browse/JDK-8157102

This improves javadoc performance across a range of workloads.

Thanks!

/Claes

Reply via email to