]> granicus.if.org Git - graphviz/commitdiff
dotgen: simplify 'makeregularend'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 26 Nov 2022 17:07:25 +0000 (09:07 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 30 Nov 2022 01:04:39 +0000 (17:04 -0800)
This function is only called with `BOTTOM` and `TOP`, so we can abbreviate it
squashing a -Wswitch-default warning.

lib/dotgen/dotsplines.c

index 25de5d47dcf850f1170668bf76978b1606597e62..fa917c5cd3a84d5cf50c764f252262695ca0a98d 100644 (file)
@@ -2175,16 +2175,11 @@ completeregularpath(path * P, edge_t * first, edge_t * last,
  */
 static boxf makeregularend(boxf b, int side, double y)
 {
-    boxf newb;
-    switch (side) {
-    case BOTTOM:
-       newb = boxfof(b.LL.x, y, b.UR.x, b.LL.y);
-       break;
-    case TOP:
-       newb = boxfof(b.LL.x, b.UR.y, b.UR.x, y);
-       break;
-    }
-    return newb;
+  assert(side == BOTTOM || side == TOP);
+  if (side == BOTTOM) {
+    return boxfof(b.LL.x, y, b.UR.x, b.LL.y);
+  }
+  return boxfof(b.LL.x, b.UR.y, b.UR.x, y);
 }
 
 #ifndef DONT_WANT_ANY_ENDPOINT_PATH_REFINEMENT