You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Regis Xu (JIRA)" <ji...@apache.org> on 2009/10/16 16:37:31 UTC
[jira] Closed: (HARMONY-6095) [classlib][luni] Performance
improvement patch for java.text.DateFormatSymbols.DateFormatSymbols(Locale)
[ https://issues.apache.org/jira/browse/HARMONY-6095?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Regis Xu closed HARMONY-6095.
-----------------------------
no response, assume OK.
> [classlib][luni] Performance improvement patch for java.text.DateFormatSymbols.DateFormatSymbols(Locale)
> --------------------------------------------------------------------------------------------------------
>
> Key: HARMONY-6095
> URL: https://issues.apache.org/jira/browse/HARMONY-6095
> Project: Harmony
> Issue Type: Improvement
> Components: Classlib
> Affects Versions: 5.0M8
> Reporter: deven you
> Assignee: Regis Xu
> Fix For: 5.0M11
>
> Attachments: DateFormatSymbolsTest.diff, hy-text.diff, hy-text_v2.diff, hy-text_v4.diff
>
> Original Estimate: 96h
> Remaining Estimate: 96h
>
> this constructor spend much time on getZoneString(), howerver it is rarely used in real world application. so I just delay loading the ZoneStrings
> the testcase is :
> import java.text.DateFormatSymbols;
> import java.util.Locale;
> public class TestDateFormatSymbols {
> public static void main(String[] args) {
> long time = System.currentTimeMillis();
> DateFormatSymbols dfs = new DateFormatSymbols(Locale.getDefault());
> time = System.currentTimeMillis() - time;
>
> System.out.println("the total time is " + time + " ms!");
> }
> }
> I have test on on Intel(R) Core(TM)2 Duo Cpu 2.4GHZ, 2.98GB Memory machine, the result is as below:
> Harmony patch before: 1125 ms
> > Harmony patched: 78 ms
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.