harmony-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tony Wu (JIRA)" <j...@apache.org>
Subject [jira] Commented: (HARMONY-3469) [classlib][archive]equalsIgnoreCase in the inner class Name could be optimized
Date Fri, 23 Mar 2007 03:56:32 GMT

    [ https://issues.apache.org/jira/browse/HARMONY-3469?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12483389

Tony Wu commented on HARMONY-3469:

Hi Alexey,
you are right, we got worse performance now. 

There are two reasons,
1.String.equalsIgnoreCase makes use of the Character.toUpper(Lower)Case. These case mapping
operations have better performance now, but they will delegate to ICU in the near future for
compatibility. ICU is designed to load all the data(unicode, timezone, locale, etc) during
its static initialization. Obviously, it will do harm to performance.
In this case, The Attributes class will be loaded when vm init, these ICU data are not useful
at that time. so, my purpose is to defer the loading of Character(ICU). 

2.this patch does have bad performance:( 
I add a new method in o.a.h.l.Util and set some condition check to make it fail fast. would
you pls try the new one? Thanks very much for your advice.

> [classlib][archive]equalsIgnoreCase in the inner class Name could be optimized
> ------------------------------------------------------------------------------
>                 Key: HARMONY-3469
>                 URL: https://issues.apache.org/jira/browse/HARMONY-3469
>             Project: Harmony
>          Issue Type: Sub-task
>          Components: Classlib
>            Reporter: Tony Wu
>         Assigned To: Alexey Petrenko
>         Attachments: harmony-3469.diff
> The code in Name's constructor guarantees that its field "name" is composed  in ASCII

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

View raw message