Remove some unneccessary assignments to argc
authorMatt Caswell <matt@openssl.org>
Thu, 21 Apr 2016 09:01:19 +0000 (10:01 +0100)
committerMatt Caswell <matt@openssl.org>
Thu, 21 Apr 2016 09:51:57 +0000 (10:51 +0100)
openssl.c and ts.c assign the value of opt_num_rest() to argc, but then
only use the value once.

Reviewed-by: Richard Levitte <levitte@openssl.org>
apps/openssl.c
apps/ts.c

index e594bd1d237c41d851f50677ac5fcd71465d3ecd..f2fb295fab5ace9460a05daad4ecb2a9c46325e8 100644 (file)
@@ -491,9 +491,8 @@ int help_main(int argc, char **argv)
             return 0;
         }
     }
-    argc = opt_num_rest();
 
-    if (argc != 0) {
+    if (opt_num_rest() != 0) {
         BIO_printf(bio_err, "Usage: %s\n", prog);
         return 1;
     }
index ff4b25eef6d3188f4d7ac51ded4bd6628c584465..cec217708eaf75f54d47b4435a85f83db810093f 100644 (file)
--- a/apps/ts.c
+++ b/apps/ts.c
@@ -319,8 +319,7 @@ int ts_main(int argc, char **argv)
             break;
         }
     }
-    argc = opt_num_rest();
-    if (mode == OPT_ERR || argc != 0)
+    if (mode == OPT_ERR || opt_num_rest() != 0)
         goto opthelp;
 
     /* Seed the random number generator if it is going to be used. */