]> granicus.if.org Git - graphviz/commitdiff
lefty: abbreviate some boolean returns in 'orderop'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 25 Sep 2021 02:30:37 +0000 (19:30 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 27 Sep 2021 14:45:28 +0000 (07:45 -0700)
cmd/lefty/aix_mods/exec.c

index 80152ca033d183799fc8b519857d715751e8289d..32556e9e21b52a93c5003f379a78fe2dc45907fc 100644 (file)
@@ -822,9 +822,7 @@ static int orderop(Tobj v1o, Ctype_t op, Tobj v2o)
     double d1, d2;
 
     if (!v1o || !v2o) {
-       if ((v1o || v2o) && op == C_NE)
-           return TRUE;
-       return FALSE;
+       return (v1o || v2o) && op == C_NE;
     }
     t1 = Tgettype(v1o), t2 = Tgettype(v2o);
     if (t1 == T_STRING && t2 == T_STRING) {
@@ -850,17 +848,17 @@ static int orderop(Tobj v1o, Ctype_t op, Tobj v2o)
     }
     switch (op) {
     case C_EQ:
-       return (r == 0) ? TRUE : FALSE;
+       return r == 0;
     case C_NE:
-       return (r != 0) ? TRUE : FALSE;
+       return r != 0;
     case C_LT:
-       return (r < 0) ? TRUE : FALSE;
+       return r < 0;
     case C_LE:
-       return (r <= 0) ? TRUE : FALSE;
+       return r <= 0;
     case C_GT:
-       return (r > 0) ? TRUE : FALSE;
+       return r > 0;
     case C_GE:
-       return (r >= 0) ? TRUE : FALSE;
+       return r >= 0;
     }
     panic1(POS, "orderop", "bad op code");
     return FALSE;              /* NOT REACHED */