gvParseArgs etc: remove tests for 'NULL' members in 'argv'

The strings in `argv` within the range of `argc` are guaranteed non-null.

diff --git a/lib/common/args.c b/lib/common/args.c
index f1c5f3b71af4c8c362a6b41b2355f300979d4223..dff892498041753d586dfbe843655a8fa51fab6a 100644 (file)
--- a/lib/common/args.c
+++ b/lib/common/args.c
@@ -17,6 +17,7 @@
  * Needs to be fixed before layout engines can be plugins.
  */
 
+#include <assert.h>
 #include <cgraph/likely.h>
 #include <ctype.h>
 #include <common/render.h>
@@ -37,7 +38,8 @@ static int neato_extra_args(int argc, char** argv) {
 
   for (i = 1; i < argc; i++) {
     arg = argv[i];
-    if (arg && *arg == '-') {
+    assert(arg != NULL);
+    if (*arg == '-') {
       switch (arg[1]) {
       case 'x' : Reduce = TRUE; break;
       case 'n':
@@ -80,7 +82,8 @@ static int memtest_extra_args(int argc, char** argv) {
 
   for (i = 1; i < argc; i++) {
     arg = argv[i];
-    if (arg && *arg == '-') {
+    assert(arg != NULL);
+    if (*arg == '-') {
       switch (arg[1]) {
       case 'm' :
         if (arg[2]) {
@@ -124,7 +127,8 @@ config_extra_args(GVC_t *gvc, int argc, char** argv)
 
   for (i = 1; i < argc; i++) {
     arg = argv[i];
-    if (arg && *arg == '-') {
+    assert(arg != NULL);
+    if (*arg == '-') {
       switch (arg[1]) {
       case 'v':
        gvc->common.verbose = 1;
@@ -242,7 +246,8 @@ static int fdp_extra_args(int argc, char** argv) {
 
   for (i = 1; i < argc; i++) {
     arg = argv[i];
-    if (arg && (*arg == '-') && (*(arg+1) == 'L')) {
+    assert(arg != NULL);
+    if ((*arg == '-') && (*(arg+1) == 'L')) {
       if (setFDPAttr (arg+2)) {
        dotneato_usage(1);
        return -1;