Bug 1338086 - Remove useless else blocks in order to reduce complexity modules/libmar/ r?mhowell draft
authorSylvestre Ledru <sledru@mozilla.com>
Thu, 09 Feb 2017 10:54:04 +0100
changeset 484417 63cd85675e8bbd936c36a670149c2b33fb3e9d85
parent 484416 9b06e634b58f6aa1d79ad0e68e9df59bca1de848
child 484418 0a132818df453e63b0c547df1b952c36ad5f48db
push id45470
push userbmo:sledru@mozilla.com
push dateWed, 15 Feb 2017 08:57:47 +0000
reviewersmhowell
bugs1338086
milestone54.0a1
Bug 1338086 - Remove useless else blocks in order to reduce complexity modules/libmar/ r?mhowell MozReview-Commit-ID: GyAIRI7Tsei
modules/libmar/tool/mar.c
--- a/modules/libmar/tool/mar.c
+++ b/modules/libmar/tool/mar.c
@@ -151,31 +151,32 @@ int main(int argc, char **argv) {
   }
 
   if (argc < 3) {
     print_usage();
     return -1;
   }
 
   while (argc > 0) {
-    if (argv[1][0] == '-' && (argv[1][1] == 'c' || 
-        argv[1][1] == 't' || argv[1][1] == 'x' || 
+    if (argv[1][0] == '-' && (argv[1][1] == 'c' ||
+        argv[1][1] == 't' || argv[1][1] == 'x' ||
         argv[1][1] == 'v' || argv[1][1] == 's' ||
         argv[1][1] == 'i' || argv[1][1] == 'T' ||
         argv[1][1] == 'r' || argv[1][1] == 'X' ||
         argv[1][1] == 'I')) {
       break;
     /* -C workingdirectory */
-    } else if (argv[1][0] == '-' && argv[1][1] == 'C') {
+    }
+    if (argv[1][0] == '-' && argv[1][1] == 'C') {
       if (chdir(argv[2]) != 0) {
         return -1;
       }
       argv += 2;
       argc -= 2;
-    } 
+    }
 #if !defined(NO_SIGN_VERIFY) && ((!defined(MAR_NSS) && defined(XP_WIN)) || \
                                  defined(XP_MACOSX))
     /* -D DERFilePath, also matches -D[index] DERFilePath
        We allow an index for verifying to be symmetric
        with the import and export command line arguments. */
     else if (argv[1][0] == '-' &&
              argv[1][1] == 'D' &&
              (argv[1][2] == (char)('0' + certCount) || argv[1][2] == '\0')) {