93066 - Shouldn't use FindCharInSet when searching for one char

Status: RESOLVED FIXED

(MailNews Core :: MIME, defect)

Description

[Daniel Bratell]

FindChar is (much) more efficient than FindCharInSet when searching for just 
one char. Still there are several misusers in the tree.

/extensions/xmlextras/base/src/nsXMLHttpRequest.cpp, line 406 -- end = 
contentType.FindCharInSet("\"", start );

/intl/strres/src/nsAcceptLang.cpp, line 114 -- PRInt32 dash = 
lc_tmp.FindCharInSet("-");

/intl/strres/src/nsAcceptLang.cpp, line 169 -- PRInt32 dash = 
acceptLang.FindCharInSet("-");

/intl/strres/src/nsStringBundle.cpp, line 446 -- PRInt32 dash = 
lc_name.FindCharInSet("-");

/xpcom/tests/windows/nsStringTest.h, line 391 -- pos = T2.FindCharInSet("A");

/xpcom/tests/windows/nsStringTest.h, line 394 -- pos=T2.RFindCharInSet("A",2);

/content/html/document/src/nsHTMLDocument.cpp, line 470 -- end = 
contentType.FindCharInSet("\"", start);

/content/xml/document/src/nsXMLDocument.cpp, line 409 -- end = 
contentType.FindCharInSet("\"", start );

/xpfe/components/search/src/nsInternetSearchService.cpp, line 4988 -- 
quoteEndOffset = resultItem.FindCharInSet(">", quoteStartOffset);

/xpfe/components/search/src/nsInternetSearchService.cpp, line 5123 -- PRInt32 
protocolOffset = site.FindCharInSet(":", 0);

/xpfe/components/search/src/nsInternetSearchService.cpp, line 5131 -- PRInt32 
slashOffset = site.FindCharInSet("/", 0);

/xpfe/components/search/src/nsInternetSearchService.cpp, line 5173 -- PRInt32 
anchorEnd = resultItem.FindCharInSet(">", quoteEndOffset);

/mailnews/base/util/nsMsgMailNewsUrl.cpp, line 604 -- PRInt32 pos = 
mAttachmentFileName.RFindCharInSet(".");

/mailnews/mime/src/mimedrft.cpp, line 1768 -- PRInt32 pos = 
contentType.FindCharInSet(";");

/mailnews/mime/src/mimemoz2.cpp, line 244 -- PRInt32 pos = 
contentType.FindCharInSet(";");

If I get the optimizations into FindCharInSet that is in another bug, that 
function will be even more optimized to handle sets larger than one char.

Comment 1

[Daniel Bratell]

There was only one offending module left. Someone must have cleaned this up
(alecf?) since this bug was filed. Will attach patch. 

alefc and sspitzer, can you r= and sr= it?

Comment 2

[Daniel Bratell]

Attachment: Replacing FindCharInSet with FindChar

Replacing 2 occurrances of FindCharInSet with FindChar because FindChar is
simpler and faster and it's confusing to use FindCharInSet when searching for a
single character.

Comment 3

[Alec Flett]

Comment on attachment 100134 [details] [diff] [review]
Replacing FindCharInSet with FindChar

yeah, I think I may have cleaned some of the others up.
sr=alecf

Comment 4

[Daniel Bratell]

Fix checked in.