[Bug 59805] LocaleUtil causes memory leak

[bu...@apache.org]

https://bz.apache.org/bugzilla/show_bug.cgi?id=59805

Javen O'Neal <on...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |INVALID
                 OS|                            |All
             Status|NEW                         |RESOLVED

--- Comment #1 from Javen O'Neal <on...@apache.org> ---
Thanks for the report.

I think Catalina is reporting a false positive here.

Looking over the short LocaleUtil class [1], there are two ThreadLocal
constants, but the thread local wrapping is necessary for the purpose of this
class: to globally set the locale and timezone.

The constants are static final, so the memory consumption doesn't increase with
time or usage.

The only change I would make to this class is making the class final.

I am closing this as invalid, unless someone provides a solution that I'm not
seeing. There should be no need to destroy and recreate threads due to
LocaleUtil.

[1]
https://svn.apache.org/viewvc/poi/trunk/src/java/org/apache/poi/util/LocaleUtil.java?view=markup

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org