]> granicus.if.org Git - graphviz/commitdiff
exsubstr: remove some unnecessary bracketing
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 24 Jul 2021 23:16:29 +0000 (16:16 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 28 Jul 2021 00:17:12 +0000 (17:17 -0700)
lib/expr/exeval.c

index eb50fbad14e9843a87e7ca3d42a837e3e4dfe171..6913482b60b2ada826ad21dc7689b9e9f69c7ba9 100644 (file)
@@ -985,11 +985,11 @@ static Extype_t exsubstr(Expr_t * ex, Exnode_t * expr, void *env)
        s = eval(ex, expr->data.string.base, env);
        len = strlen(s.string);
        i = eval(ex, expr->data.string.pat, env);
-       if ((i.integer < 0) || (len < i.integer))
+       if (i.integer < 0 || len < i.integer)
                exerror("illegal start index in substr(%s,%d)", s.string, i.integer);
        if (expr->data.string.repl) {
                l = eval(ex, expr->data.string.repl, env);
-               if ((l.integer < 0) || (len - i.integer < l.integer))
+               if (l.integer < 0 || len - i.integer < l.integer)
            exerror("illegal length in substr(%s,%d,%d)", s.string, i.integer, l.integer);
        } else
                l.integer = len - i.integer;