]> granicus.if.org Git - graphviz/commitdiff
gvplugin.c: remove unnecessary parens
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 4 Nov 2021 15:16:16 +0000 (08:16 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 21 Dec 2021 02:40:39 +0000 (18:40 -0800)
lib/gvc/gvplugin.c

index ecb79aac5c6fc3cd85067389d7e026c14fef6cfe..7ea4614f83e2a0cef95e9d7879a0bfa85c9616a2 100644 (file)
@@ -88,7 +88,7 @@ bool gvplugin_install(GVC_t *gvc, api_t api, const char *typestr, int quality,
         *p = '\0';
 
     /* point to the beginning of the linked list of plugins for this api */
-    pnext = &(gvc->apis[api]);
+    pnext = &gvc->apis[api];
 
     /* keep alpha-sorted and insert new duplicates ahead of old */
     while (*pnext) {
@@ -97,7 +97,7 @@ bool gvplugin_install(GVC_t *gvc, api_t api, const char *typestr, int quality,
             *p = '\0';
         if (strcmp(pins, pnxt) <= 0)
             break;
-        pnext = &((*pnext)->next);
+        pnext = &(*pnext)->next;
     }
 
     /* keep quality sorted within type and insert new duplicates ahead of old */
@@ -109,7 +109,7 @@ bool gvplugin_install(GVC_t *gvc, api_t api, const char *typestr, int quality,
             break;
         if (quality >= (*pnext)->quality)
             break;
-        pnext = &((*pnext)->next);
+        pnext = &(*pnext)->next;
     }
 
     plugin = GNEW(gvplugin_available_t);
@@ -140,10 +140,10 @@ static void gvplugin_activate(GVC_t * gvc, api_t api, const char *typestr,
     pnext = gvc->apis[api];
 
     while (pnext) {
-        if ((strcasecmp(typestr, pnext->typestr) == 0)
-            && (strcasecmp(name, pnext->package->name) == 0)
-            && (pnext->package->path != 0)
-            && (strcasecmp(plugin_path, pnext->package->path) == 0)) {
+        if (strcasecmp(typestr, pnext->typestr) == 0
+            && strcasecmp(name, pnext->package->name) == 0
+            && pnext->package->path != 0
+            && strcasecmp(plugin_path, pnext->package->path) == 0) {
             pnext->typeptr = typeptr;
             return;
         }
@@ -191,7 +191,7 @@ gvplugin_library_t *gvplugin_library_load(GVC_t * gvc, char *path)
     }
     hndl = lt_dlopen(p);
     if (!hndl) {
-        if ((stat(p, &sb)) == 0) {
+        if (stat(p, &sb) == 0) {
             agerr(AGWARN, "Could not load \"%s\" - %s\n", p, "It was found, so perhaps one of its dependents was not.  Try ldd.");
         }
         else {
@@ -232,7 +232,7 @@ gvplugin_library_t *gvplugin_library_load(GVC_t * gvc, char *path)
         return NULL;
     }
     free(sym);
-    return (gvplugin_library_t *) (ptr);
+    return (gvplugin_library_t *)ptr;
 #else
     agerr(AGERR, "dynamic loading not available\n");
     return NULL;
@@ -308,8 +308,8 @@ gvplugin_available_t *gvplugin_load(GVC_t * gvc, api_t api, const char *str)
         if (!reqpkg || strcmp(reqpkg, pnext->package->name) == 0) {
             // found with no packagename constraints, or with required matching packagename
 
-            if (dep && (apidep != api)) /* load dependency if needed, continue if can't find */
-                if (!(gvplugin_load(gvc, apidep, dep)))
+            if (dep && apidep != api) /* load dependency if needed, continue if can't find */
+                if (!gvplugin_load(gvc, apidep, dep))
                     continue;
             break;
         }