https://issues.apache.org/bugzilla/show_bug.cgi?id=54147
Daniel Trebbien <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #1 from Daniel Trebbien <[email protected]> --- 1. Rather than moving and renaming the encodedata() method, why not make the necessary changes in place? This way, history is better preserved. 2. The new implementation is not correct. It writes out "]]]]><![CDATA[>" as soon as it sees "]]", but this means that "]]>" is replaced with "]]]]><![CDATA[>>". What you could do is write out "]]]]><![CDATA[" instead. Though, this slightly changes the behavior (i.e. instead of escaping "]]>", it actually escapes "]]") and the JavaDoc comment would need to be updated. 3. Because characters are written individually in the new code, there is likely a negative impact on performance. -- You are receiving this mail because: You are the assignee for the bug.
