[ https://issues.apache.org/jira/browse/LANG-421?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12584648#action_12584648 ]
[EMAIL PROTECTED] edited comment on LANG-421 at 4/2/08 8:51 AM: ----------------------------------------------------------- The JLS reference is http://java.sun.com/docs/books/jls/second_edition/html/lexical.doc.html#101083 (3.10.5 String Literals) was (Author: [EMAIL PROTECTED]): The JLS reference is http://java.sun.com/docs/books/jls/second_edition/html/lexical.doc.html#101083 > StringEscapeUtils.escapeJava(String) escapes '/' characters > ----------------------------------------------------------- > > Key: LANG-421 > URL: https://issues.apache.org/jira/browse/LANG-421 > Project: Commons Lang > Issue Type: Bug > Affects Versions: 2.4 > Reporter: Steve Hanmann > Priority: Blocker > Attachments: StringEscapeUtilsTest.java > > > Commons Lang 2.4 StringEscapeUtils.escapeJava(String) now escapes '/' > characters, which is not a valid "escapable" character in Java strings. I > haven't tried the other Java escape/unescape methods to see if they have a > similar problem, or that only Java "escapable" characters are escaped by > escapeJava(String). > This bug may have appeared as an unintended side-effect of the fix for > LANG-363. > Also the javadoc for escapeJava is now a little off, in that '/' should now > be included in the sentence describing the differences between Java and > Javascript strings, with respect to escaping rules. > The following is a JUnit3 test demonstrating the bug. > import junit.framework.TestCase; > import org.apache.commons.lang.StringEscapeUtils; > public class StringEscapeUtilsTest extends TestCase { > public void testEscapeJavaWithSlash() { > final String input = "String with a slash (/) in it"; > > final String expected = input; > final String actual = StringEscapeUtils.escapeJava( input ); > /** > * In 2.4 StringEscapeUtils.escapeJava(String) escapes '/' characters, > * which are not a valid character to escape in a Java string. > */ > assertEquals( expected, actual ); > } > } -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.