]> granicus.if.org Git - graphviz/commitdiff
remove some unnecessary bracketing in merge_trapezoids()
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 11 Oct 2020 16:55:24 +0000 (09:55 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 21 Oct 2020 00:27:25 +0000 (17:27 -0700)
This has no effect on functionality, but makes the resulting code less noisy and
easier to read.

lib/ortho/trapezoid.c

index cb3b60407fba49b81199d79253e7d9db6c6a6c6e..985cd935085f9c8e0e31e8c4b292295226cf79fb 100644 (file)
@@ -358,19 +358,19 @@ merge_trapezoids (int segnum, int tfirst, int tlast, int side, trap_t* tr,
 
   /* First merge polys on the LHS */
   t = tfirst;
-  while ((t > 0) && _greater_than_equal_to(&tr[t].lo, &tr[tlast].lo))
+  while (t > 0 && _greater_than_equal_to(&tr[t].lo, &tr[tlast].lo))
     {
       if (side == S_LEFT)
-       cond = ((((tnext = tr[t].d0) > 0) && (tr[tnext].rseg == segnum)) ||
-               (((tnext = tr[t].d1) > 0) && (tr[tnext].rseg == segnum)));
+       cond = ((tnext = tr[t].d0) > 0 && tr[tnext].rseg == segnum) ||
+               ((tnext = tr[t].d1) > 0 && tr[tnext].rseg == segnum);
       else
-       cond = ((((tnext = tr[t].d0) > 0) && (tr[tnext].lseg == segnum)) ||
-               (((tnext = tr[t].d1) > 0) && (tr[tnext].lseg == segnum)));
+       cond = ((tnext = tr[t].d0) > 0 && tr[tnext].lseg == segnum) ||
+               ((tnext = tr[t].d1) > 0 && tr[tnext].lseg == segnum);
 
       if (cond)
        {
-         if ((tr[t].lseg == tr[tnext].lseg) &&
-             (tr[t].rseg == tr[tnext].rseg)) /* good neighbours */
+         if (tr[t].lseg == tr[tnext].lseg &&
+             tr[t].rseg == tr[tnext].rseg) /* good neighbours */
            {                                 /* merge them */
              /* Use the upper node as the new node i.e. t */