[ 
https://issues.apache.org/jira/browse/LANG-474?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Henri Yandell closed LANG-474.
------------------------------

    Resolution: Fixed

StrMatch inner class fields now final. Closing.

svn ci -m "Making inner class private fields final per LANG-474"
Sending        src/java/org/apache/commons/lang/text/StrMatcher.java
Transmitting file data .
Committed revision 803006.

> Fixes for thread safety
> -----------------------
>
>                 Key: LANG-474
>                 URL: https://issues.apache.org/jira/browse/LANG-474
>             Project: Commons Lang
>          Issue Type: Bug
>            Reporter: Sebb
>            Priority: Minor
>             Fix For: 3.0
>
>
> org.apache.commons.lang.exception.NestableDelegate
> - the static boolean fields should probably be made volatile to ensure that 
> they are visible across threads
> If not, the Javadoc should be updated to note that changing the setting 
> renders the class not thread-safe.
> - the "nestable" field should be declared final
> org.apache.commons.lang.exception.NestableError / NestableRuntimeException / 
> NestableException
> - Fields "cause" and "delegate" should be made final to improve thread-safety
> The following fields could be made final:
> org.apache.commons.lang.text.ExtendedMessageFormat.registry
> org.apache.commons.lang.text.StrMatcher - class variables in nested classes 
> could be made final
> org.apache.commons.lang.CharSet.set
> Also:
> org.apache.commons.lang.IntHashMap.Entry - the field "key" is never used

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to