Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fixing bustage. I didn't realize what a common (and mistaken) pattern it was to compare a |nsString| to |nsnull|. Better to say |IsEmpty()|.

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@234729 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
scc%mozilla.org
2007-09-06 21:57:34 +00:00
parent 15cb7e375e
commit 19073dcf2d
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
mozilla/suite/browser/src/nsLocalSearchService.cpp
View File

@@ -335,7 +335,7 @@ LocalSearchDataSource::doMatch(nsIRDFLiteral *literal, const nsString &matchMeth
{
PRBool found = PR_FALSE;
if ((nsnull == literal) || (nsnull == matchMethod) || (nsnull == matchText))
if ((nsnull == literal) || matchMethod.IsEmpty() || matchText.IsEmpty())
return(found);
const PRUnichar *str = nsnull;