search

damnhandy / Handy-URI-Templates

A Java URI Template processor implementing RFC6570
https://damnhandy.github.io/Handy-URI-Templates/
Other
203 stars 37 forks source link

Replace SimpleDateFormat with Joda Time to Improve Performance #30

Closed asaph closed 9 years ago

asaph commented 9 years ago

The poor performance of java.text.SimpleDateFormat is well documented. Currently, every instantiation of UriTemplate creates an instance of SimpleDateFormat, whether it's needed or not. This pull request replaces the functionality of SimpleDateFormat with analogous functionality from Joda-Time which improves performance.

Note: A couple of methods and one field that exposed DateFormat and SimpleDateFormat, and that were part of this library's public interface, were marked as deprecated. I thought it better that the library shouldn't expose the details of its date formatting implementation. Incidentally, the deprecated methods didn't have test coverage (which this PR also adds). The deprecated items should be kept around until the next major version in order to maintain backwards compatibility with existing clients.

buildhive commented 9 years ago

Ryan J. McDonough » Handy-URI-Templates #122 SUCCESS This pull request looks good (what's this?)