[ http://issues.apache.org/jira/browse/LANG-260?page=all ]
Henri Yandell resolved LANG-260:
--------------------------------
Resolution: Fixed
svn ci -m "Exposing the unescapeXml and escapeXml methods that take Writers -
LANG-260. A recent thread
did point out that there are problems with the concept of escaping Xml as a
single method, and instead it needs to be an xml parser that escapes the body an
d the attribute content differently - however we're obviously not there yet and
I don't think making the existing 80/20 good enough code more usable hurts.
" src/java/org/apache/commons/lang/StringEscapeUtils.java
src/test/org/apache/commons/lang/StringEscapeUtilsTest.java
Sending src/java/org/apache/commons/lang/StringEscapeUtils.java
Sending src/test/org/apache/commons/lang/StringEscapeUtilsTest.java
Transmitting file data ..
Committed revision 412009.
> StringEscapeUtils should expose escape*() methods taking Writer argument
> ------------------------------------------------------------------------
>
> Key: LANG-260
> URL: http://issues.apache.org/jira/browse/LANG-260
> Project: Commons Lang
> Type: Improvement
> Environment: N/A
> Reporter: Norbert Reilly
> Fix For: 2.2
>
> StringEscapeUtils.escapeXML(String) is the only exposed variant, but the
> package private Entities.XML class it uses under the covers has a variant
> taking a Writer. The StringEscapeUtils.escapeXML(Writer, String) variant
> would be much more efficient when incrementally escaping values and adding
> them one-by-one and could seemingly be exposed with very little effort. Ditto
> for the unescapeXML() case, and presumably other styles of escape/unescape
> (like HTML) that use the Entities class under the covers.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
