add case-insensitive wstring compare to nsICaseConversion

RESOLVED FIXED in mozilla0.9.6

Status

()

Core
Internationalization
P2
normal
RESOLVED FIXED
17 years ago
17 years ago

People

(Reporter: Alec Flett, Assigned: Alec Flett)

Tracking

({intl})

Trunk
mozilla0.9.6
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
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)

Updated

17 years ago
Blocks: 100214
(Assignee)

Comment 1

17 years ago
Created attachment 52989 [details] [diff] [review]
add CaseCompare()
(Assignee)

Updated

17 years ago
Priority: -- → P2
Target Milestone: --- → mozilla0.9.6
(Assignee)

Updated

17 years ago
No longer blocks: 100214
(Assignee)

Updated

17 years ago
Blocks: 100214

Comment 2

17 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

17 years ago
Not sure I like the name "CaseCompare", "CaseInsensitiveCompare" would be more
appropriate. Other than that, r=jag
CompareIgnorngCase?  Definitely not "CaseCompare", though.  sr=shaver
(Assignee)

Comment 5

17 years ago
checked in as "CaseInsensitiveCompare" - thanks folks
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
(Assignee)

Comment 6

17 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.