104122 - add case-insensitive wstring compare to nsICaseConversion

Status: RESOLVED FIXED

(Core :: Internationalization, defect, P2)

Description

[Alec Flett]

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

Comment 1

[Alec Flett]

Attachment: add CaseCompare()

Comment 2

[Andreas Becker]

Switching qa contact to [e-mail], who is qa contact of bug 100214. Please 
re-assign further as appropriate.

Comment 3

[jag (Peter Annema)]

Not sure I like the name "CaseCompare", "CaseInsensitiveCompare" would be more
appropriate. Other than that, r=jag

Comment 4

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

CompareIgnorngCase?  Definitely not "CaseCompare", though.  sr=shaver

Comment 5

[Alec Flett]

checked in as "CaseInsensitiveCompare" - thanks folks

Comment 6

[Alec Flett]

by the way, there are no consumers of this until I finish with bug 100214