Closed
Bug 104122
Opened 24 years ago
Closed 24 years ago
add case-insensitive wstring compare to nsICaseConversion
Categories
(Core :: Internationalization, defect, P2)
Tracking
()
RESOLVED
FIXED
mozilla0.9.6
People
(Reporter: alecf, Assigned: alecf)
References
Details
(Keywords: intl)
Attachments
(1 file)
|
4.62 KB,
patch
|
Details | Diff | Splinter Review |
Broken out of bug 100214:
We need a fast way of case-insensitively comparing strings (or chunks of them)
without incuring a two virtual method calls to nsICaseConversion::ToUpper() for
every character compared.
One option would be to upcase both strings being compared, but that would
require a buffer to hold each string, and that also incurs unnecessary costs of
maintaining this buffer, etc.
And so, I propose we add a method to nsICaseConversion, CaseCompare() which does
this comparison for us.
patches forthcoming
|
Assignee | |
Comment 1•24 years ago
|
||
|
|
Assignee | |
Updated•24 years ago
|
Priority: -- → P2
Target Milestone: --- → mozilla0.9.6
|
|
||
Comment 2•24 years ago
|
||
Switching qa contact to scc@mozilla.org, who is qa contact of bug 100214. Please
re-assign further as appropriate.
Keywords: intl
QA Contact: andreasb → scc
|
||
Comment 3•24 years ago
|
||
Not sure I like the name "CaseCompare", "CaseInsensitiveCompare" would be more
appropriate. Other than that, r=jag
|
||
Comment 4•24 years ago
|
||
CompareIgnorngCase? Definitely not "CaseCompare", though. sr=shaver
|
|
Assignee | |
Comment 5•24 years ago
|
||
checked in as "CaseInsensitiveCompare" - thanks folks
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 6•24 years ago
|
||
by the way, there are no consumers of this until I finish with bug 100214
You need to log in
before you can comment on or make changes to this bug.
Description
•