RFR: 7113878: LogManager - namedLoggers should be ConcurrentHashMap instead of Hashtable

David Holmes david.holmes at oracle.com
Thu Mar 26 12:28:39 UTC 2015


Hi Daniel,

On 26/03/2015 10:08 PM, Daniel Fuchs wrote:
> Please find below a trivial fix for
>
>
> 7113878: LogManager - namedLoggers should be ConcurrentHashMap
>           instead of Hashtable

As you say in the bug report, now that the map is always accessed within 
synchronized code this serves no purpose. The map not only doesn't need 
to be concurrent, it doesn't even need to be thread-safe! So why not 
replace with a simple HashMap?

David

> https://bugs.openjdk.java.net/browse/JDK-7113878
>
> http://cr.openjdk.java.net/~dfuchs/webrev_7113878/webrev.00
>
> best regards,
>
> -- daniel



More information about the core-libs-dev mailing list