[ https://issues.apache.org/jira/browse/LANG-1308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15799692#comment-15799692 ]
Gary Gregory commented on LANG-1308: ------------------------------------ An interesting alternative would be to use a service loader for getting various escapers and to put the CSV code in the Commons CSV component which deals with all manners of CSV complications. > Move CvsTranslators out of StringEscapeUtils and make them DRY > -------------------------------------------------------------- > > Key: LANG-1308 > URL: https://issues.apache.org/jira/browse/LANG-1308 > Project: Commons Lang > Issue Type: Improvement > Components: lang.*, lang.text.translate.* > Affects Versions: 3.5 > Reporter: Jarosław Strzelecki > Priority: Minor > > CvsEscaper/CvsUnescaper are currently inner classes of StringEscapeUtils and > it does not seem like it is a proper place for them. > Since they are package-private, they can be safely moved lang.text.translate > package. > Moreover it's possible make them more DRY by introducing SinlgePassTranslator > as suggested in TODO. -- This message was sent by Atlassian JIRA (v6.3.4#6332)