[Mono-bugs] [Bug 656139] New: ResourceSet is extremely slow when using ignoreCase to fetch resource strings

bugzilla_noreply at novell.com bugzilla_noreply at novell.com
Fri Nov 26 13:30:41 EST 2010


https://bugzilla.novell.com/show_bug.cgi?id=656139

https://bugzilla.novell.com/show_bug.cgi?id=656139#c0


           Summary: ResourceSet is extremely slow when using ignoreCase to
                    fetch resource strings
    Classification: Mono
           Product: Mono: Class Libraries
           Version: 2.8.x
          Platform: Macintosh
        OS/Version: Mac OS X 10.6
            Status: NEW
          Severity: Normal
          Priority: P5 - None
         Component: CORLIB
        AssignedTo: mono-bugs at lists.ximian.com
        ReportedBy: curtis.wensley at gmail.com
         QAContact: mono-bugs at lists.ximian.com
          Found By: ---
           Blocker: ---


Description of Problem:
When retrieving resource strings from a ResourceManager with IgnoreCase = true,
it goes extremely slow when there are thousands of strings.  This is due to the
ResourceSet iterating through all strings in the hashtable when ignoreCase=true
when a string is not found.

Proposed patch to ResourceSet: 
https://github.com/cwensley/mono/commit/8e88b688d7f418f9f69437022f8291012e60e034

Steps to reproduce the problem:
1. Create a resource manager/set with thousands of strings
2. get string resources


Actual Results:
Current implementation is very slow looping through all strings

Expected Results:
Should use a separate hashtable (with ignoreCase) to lookup strings.

How often does this happen? 
every time

Additional Information:
Using ignoreCase is important for systems that have user-entered resource
strings (e.g. in the database) so that users don't have to worry about case.

-- 
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.


More information about the mono-bugs mailing list