My time is limited these days but I should be able to shepherd/RM a 2.4.1. I'll look up the laundry list of stuff that needs to be addressed. Feel free to suggest too ;)
Thank you, Gary > -----Original Message----- > From: Henri Yandell [mailto:[EMAIL PROTECTED] > Sent: Tuesday, April 08, 2008 9:08 PM > To: Commons Developers List > Subject: Re: [jira] Resolved: (LANG-421) StringEscapeUtils.escapeJava(String) > escapes '/' characters > > Sure - if you're up for being the RM :) > > I've a week and a half of freedom left - I suspect it would take a bit > longer to get a 2.4.1 out. > > Hen > > On Wed, Apr 2, 2008 at 2:16 PM, Gary Gregory > <[EMAIL PROTECTED]> wrote: > > Arg, is there any interest in pushing out a 2.4.1 for this? > > > > Thank you, > > Gary > > > > > -----Original Message----- > > > From: Gary Gregory (JIRA) [mailto:[EMAIL PROTECTED] > > > Sent: Wednesday, April 02, 2008 9:09 AM > > > To: Gary Gregory > > > Subject: [jira] Resolved: (LANG-421) StringEscapeUtils.escapeJava(String) > escapes > > > '/' characters > > > > > > > > > [ https://issues.apache.org/jira/browse/LANG- > > > 421?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] > > > > > > Gary Gregory resolved LANG-421. > > > ------------------------------- > > > > > > Resolution: Fixed > > > Fix Version/s: Nightly Builds > > > 3.0 > > > > > > Fixed. Added unit test method based on attachment provided. > > > > > > > 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 > > > > Assignee: Gary Gregory > > > > Priority: Blocker > > > > Fix For: 3.0, Nightly Builds > > > > > > > > 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. > > > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
