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

Niall Pemberton updated CODEC-55:
---------------------------------

    Attachment: CODEC-55-Wrapper-Implementations.patch

An alternative solution is to provide wrappers that only expose the 
Encoder/Decoder methods.

Attaching CODEC-55-Wrapper-Implementations.patch which contains:

 * StringEncoderWrapper - wrapper for StringEncoder implementations - can be 
used to resolve DoubleMetaphone, Metaphone, RefinedSoundex and Soundex thread 
safety issues
 * StringCodecWrapper  - wrapper for StringEncoder and StringDecoder 
implementations - can be used to resolve QCodec thread safety issue (making 
charset final has already been committed).

The other issue was with the protected fields in URLCodec - changing them to 
final would break binary compatibility but also its less of an issue IMO since 
URLCodec used directly will not have an issue as those fields are not exposed - 
its only implementations that extend from URLCodec that could have problems.

> make all "business" method implementations of public API thread safe 
> ---------------------------------------------------------------------
>
>                 Key: CODEC-55
>                 URL: https://issues.apache.org/jira/browse/CODEC-55
>             Project: Commons Codec
>          Issue Type: Wish
>            Reporter: Qingtian Wang
>         Attachments: CODEC-55-Wrapper-Implementations.patch, 
> concurrentCodecs.diff, concurrentQDiff.diff, urlcodec.patch
>
>
> Maybe most of the implementations are already thread safe. Just such that 
> codec can say so in general...

-- 
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