From 19073dcf2d625338d14ae0492c83281caae213fe Mon Sep 17 00:00:00 2001 From: "scc%mozilla.org" Date: Thu, 6 Sep 2007 21:57:34 +0000 Subject: [PATCH] fixing bustage. I didn't realize what a common (and mistaken) pattern it was to compare a |nsString| to |nsnull|. Better to say |IsEmpty()|. git-svn-id: svn://10.0.0.236/trunk@234729 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/suite/browser/src/nsLocalSearchService.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mozilla/suite/browser/src/nsLocalSearchService.cpp b/mozilla/suite/browser/src/nsLocalSearchService.cpp index eb5293b099d..536db3e8bb2 100755 --- a/mozilla/suite/browser/src/nsLocalSearchService.cpp +++ b/mozilla/suite/browser/src/nsLocalSearchService.cpp @@ -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;