Bug 1338086 - Remove useless else blocks in order to reduce complexity in toolkit/components/remote/ r?ehsan draft
authorSylvestre Ledru <sledru@mozilla.com>
Thu, 09 Feb 2017 10:57:00 +0100
changeset 484411 abd4456d4ea9027b64a5330fc23711679bd7a8a9
parent 484410 dcb1fd34816adaa606f6dcafc504c51498b5c769
child 484412 2cc9e578b620f057c2c69006508fffc183deef59
push id45470
push userbmo:sledru@mozilla.com
push dateWed, 15 Feb 2017 08:57:47 +0000
reviewersehsan
bugs1338086
milestone54.0a1
Bug 1338086 - Remove useless else blocks in order to reduce complexity in toolkit/components/remote/ r?ehsan MozReview-Commit-ID: 2xcvOyxtDsh
toolkit/components/remote/nsXRemoteService.cpp
--- a/toolkit/components/remote/nsXRemoteService.cpp
+++ b/toolkit/components/remote/nsXRemoteService.cpp
@@ -216,17 +216,17 @@ nsXRemoteService::HandleNewProperty(XID 
                      sMozResponseAtom, XA_STRING,
                      8, PropModeReplace,
                      (const unsigned char *)response,
                      strlen (response));
     XFree(data);
     return true;
   }
 
-  else if (aChangedAtom == sMozResponseAtom) {
+  if (aChangedAtom == sMozResponseAtom) {
     // client accepted the response.  party on wayne.
     return true;
   }
 
   else if (aChangedAtom == sMozLockAtom) {
     // someone locked the window
     return true;
   }