No bug - Remove always-true comparison.
authorMs2ger <ms2ger@gmail.com>
Sat, 24 Mar 2012 09:18:09 +0100
changeset 93446 f1c738169b20a03dbd49263043f574f2634df2b9
parent 93445 df1f94b2bdee149285c9f30ec7e62ff27110fe97
child 93447 53ed280cc94bed37aeed7a7057dd930e3be82863
push id886
push userlsblakk@mozilla.com
push dateMon, 04 Jun 2012 19:57:52 +0000
treeherdermozilla-beta@bbd8d5efd6d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
milestone14.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
No bug - Remove always-true comparison.
netwerk/base/src/nsIOService.cpp
--- a/netwerk/base/src/nsIOService.cpp
+++ b/netwerk/base/src/nsIOService.cpp
@@ -1183,17 +1183,17 @@ nsIOService::TrackNetworkLinkStatusForOf
     return SetOffline(!isUp);
 }
 
 NS_IMETHODIMP
 nsIOService::EscapeString(const nsACString& aString,
                           PRUint32 aEscapeType,
                           nsACString& aResult)
 {
-  NS_ENSURE_ARG_RANGE(aEscapeType, 0, 4);
+  NS_ENSURE_ARG_MAX(aEscapeType, 4);
 
   nsCAutoString stringCopy(aString);
   nsCString result;
 
   if (!NS_Escape(stringCopy, result, (nsEscapeMask) aEscapeType))
     return NS_ERROR_OUT_OF_MEMORY;
 
   aResult.Assign(result);