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

James Carman commented on CODEC-55:
-----------------------------------

This all seems quite contrived to me.  First of all, the classes that you're 
adding the createThreadSafeCodec method to aren't all "codecs" (RefinedSoudex 
is an Encoder, but not a Decoder).  Also, this solution doesn't fix the problem 
that the user (from the use case of our discussion on the user list) still has 
to have multiple instances of the Encoder/Decoder to account for the 
combinations of properties that they need (encodeBlanks and charset for QCodec) 
or just instantiate them on-the-fly.



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