Bug 1338086 - Remove useless else blocks in order to reduce complexity in modules/libpref/ r?froydnj
MozReview-Commit-ID: EGiX5NWdj6r
--- a/modules/libpref/prefapi.cpp
+++ b/modules/libpref/prefapi.cpp
@@ -476,23 +476,21 @@ pref_CompareStrings(const void *v1, cons
{
char *s1 = *(char**) v1;
char *s2 = *(char**) v2;
if (!s1)
{
if (!s2)
return 0;
- else
- return -1;
+ return -1;
}
- else if (!s2)
+ if (!s2)
return 1;
- else
- return strcmp(s1, s2);
+ return strcmp(s1, s2);
}
bool PREF_HasUserPref(const char *pref_name)
{
if (!gHashTable)
return false;
PrefHashEntry *pref = pref_HashTableLookup(pref_name);