On 21/06/2013 16:04, Brian Burkhalter wrote:

On Jun 21, 2013, at 2:49 AM, Alan Bateman wrote:

One part that might need attention is getRadixConversionCache as I could imagine cases where the synchronization could lead to contention. Have you (or Alan) considered alternatives that would limit the synchronization to only when the powers cache needs to be expanded?

I have not but will do so. I cannot speak for Alan E.

"static synchronized" always jumps out but since every toString uses it when beyond the threshold then it would be good to eliminate. You're doing great work to sequence all these patches and if fixing this now delays the plan then it would be fine to address it later.

-Alan

Reply via email to