From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 14 Aug 2021 21:35:58 +0000 (-0700)
Subject: gvmap: remove some unnecessary bracketing
X-Git-Tag: 2.49.0~9^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=c2b442d6efc1cf9a7b92a80f8435b7453c4cea02;p=graphviz

gvmap: remove some unnecessary bracketing
---

diff --git a/cmd/gvmap/gvmap.c b/cmd/gvmap/gvmap.c
index 23023d090..0f33707fc 100644
--- a/cmd/gvmap/gvmap.c
+++ b/cmd/gvmap/gvmap.c
@@ -186,7 +186,7 @@ init(int argc, char **argv, params_t* pm)
   while ((c = getopt(argc, argv, ":evODQko:m:s:r:p:c:C:l:b:g:t:a:h:z:d:?")) != -1) {
     switch (c) {
     case 'm':
-      if ((sscanf(optarg,"%lf",&s) > 0) && (s != 0)){
+      if (sscanf(optarg, "%lf", &s) > 0 && s != 0) {
 	    pm->bbox_margin[0] =  pm->bbox_margin[1] = s;
       } else {
         usage(cmd, 1);
@@ -196,34 +196,35 @@ init(int argc, char **argv, params_t* pm)
       pm->clusterMethod = CLUSTERING_MQ;
       break;
     case 's':
-      if ((sscanf(optarg,"%lf",&s) > 0)){
+      if (sscanf(optarg, "%lf", &s) > 0) {
         pm->shore_depth_tol = s;
       } else {
         usage(cmd,1);
       }
       break;
     case 'h':
-      if ((sscanf(optarg,"%d",&v) > 0)){
+      if (sscanf(optarg, "%d", &v) > 0) {
         pm->nedgep = MAX(0, v);
-      } else if (!strncmp(optarg, HLPFX, N_HLPFX) && (sscanf(optarg+N_HLPFX,"%d",&v) > 0)) {
+      } else if (!strncmp(optarg, HLPFX, N_HLPFX) &&
+                 sscanf(optarg + N_HLPFX, "%d", &v) > 0) {
         pm->highlight_cluster = MAX(0, v);
       } else {
         usage(cmd,1);
       }
       break;
      case 'r':
-      if ((sscanf(optarg,"%d",&r) > 0)){
+      if (sscanf(optarg, "%d", &r) > 0) {
         pm->nrandom = r;
       }
       break;
     case 't':
-      if ((sscanf(optarg,"%d",&r) > 0) && r > 0){
+      if (sscanf(optarg, "%d", &r) > 0 && r > 0) {
         pm->improve_contiguity_n = r;
       }
       break;
     case 'p':
       pm->show_points = 1;
-      if ((sscanf(optarg,"%d",&r) > 0)){
+      if (sscanf(optarg, "%d", &r) > 0) {
         pm->show_points = MIN(3, r);
       }
       break;
@@ -246,7 +247,7 @@ init(int argc, char **argv, params_t* pm)
       pm->color_optimize = false;
       break;
     case 'a':
-      if ((sscanf(optarg,"%d",&r) > 0)){
+      if (sscanf(optarg, "%d", &r) > 0) {
 	    pm->nart = r;
       } else {
 	    usage(cmd,1);
@@ -255,7 +256,8 @@ init(int argc, char **argv, params_t* pm)
     case 'c':
       if (sscanf(optarg,"_opacity=%2s", stmp) > 0 && strlen(stmp) == 2){
         pm->opacity = stmp;
-      } else if ((sscanf(optarg,"%d",&r) > 0) && r >= COLOR_SCHEME_NONE && r <= COLOR_SCHEME_GREY){
+      } else if (sscanf(optarg, "%d", &r) > 0 && r >= COLOR_SCHEME_NONE &&
+                 r <= COLOR_SCHEME_GREY) {
         pm->color_scheme = r;
       } else if (knownColorScheme(optarg)) {
         pm->color_scheme = COLOR_SCHEME_NONE;
@@ -273,7 +275,7 @@ init(int argc, char **argv, params_t* pm)
         pm->seed = v;
       break;
     case 'C':
-      if (!((sscanf(optarg,"%d",&v) > 0) && v >= 0)){
+      if (!(sscanf(optarg, "%d", &v) > 0 && v >= 0)) {
         usage(cmd,1);
       }
       else