[ 
https://issues.apache.org/jira/browse/LANG-462?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13211396#comment-13211396
 ] 

Felix Müller commented on LANG-462:
-----------------------------------

Seems that I should have installed a pure JDK 5 on my machine and run the maven 
build with java-1.5 profile first: As you can see on the CI server, I forgot 
one java 6 dependency. I also fixed this with the next attached patch.

With the use of JDK 5 the FastDateParser has a really curious problem. One of 
the test patterns causes the parser to end up in an infinite loop. I debugged 
through the code: In JDK 5 the Pattern.compile method seems to be buggy. You 
can reproduce the problem when you look into FastDatePrinterTest line 115/116 
(with the merged patch) and run the test class FastDateFormat_PrinterTest. To 
secure the test I added a timeout of 20 seconds.

The attached patch consists of the fully Java 5 compatible classes (I 
successfully run "mvn clean package -Pjava-1.5"), some minor clean up and the 
semi-fixed test.

I don't know if it would be good to release a class where we cannot say under 
what circumstances it will crash or end up in an infinite loop.
                
> FastDateFormat supports parse
> -----------------------------
>
>                 Key: LANG-462
>                 URL: https://issues.apache.org/jira/browse/LANG-462
>             Project: Commons Lang
>          Issue Type: New Feature
>          Components: lang.time.*
>            Reporter: Franz Wong
>             Fix For: 3.2
>
>         Attachments: DateParser.patch, LANG-462-FormatCache.patch, 
> LANG-462-Hen.patch, LANG-462_buildfix.patch, 
> LANG-462_buildfix_pending_java6_issues.patch, UseFormatCache.patch, 
> lang462.patch, with_interfaces.patch, with_interfaces2.patch, 
> with_updated_tests.patch
>
>
> Currently FastDateFormat only supports formatting the ISO8601 time zone, 
> however, it doesn't support parsing such string to Date.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to