]> granicus.if.org Git - graphviz/commitdiff
expr: [nfc] remove unnecessary parens and dereference of 'matchf'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 26 Feb 2022 19:08:03 +0000 (11:08 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 27 Feb 2022 22:57:01 +0000 (14:57 -0800)
The C compiler knows how to dereference and call through a function pointer
without needing this manual instruction.

lib/expr/exeval.c

index f9f14cd7d7f26f625ff4781379fdd620731a8d1e..2762dd366348b577b8d3ab7ac96e8e11f95576d2 100644 (file)
@@ -1303,7 +1303,10 @@ eval(Expr_t* ex, Exnode_t* expr, void* env)
                                                        break;
                                                continue;
                                        case STRING:
-                                               if ((ex->disc->version >= 19981111L && ex->disc->matchf) ? (*ex->disc->matchf)(ex, x, (*t)->string, expr->data.operand.left, v.string, env, ex->disc) : strmatch((*t)->string, v.string))
+                                               if ((ex->disc->version >= 19981111L && ex->disc->matchf)
+                                                     ? ex->disc->matchf(ex, x, (*t)->string, expr->data.operand.left,
+                                                                        v.string, env, ex->disc)
+                                                     : strmatch((*t)->string, v.string))
                                                        break;
                                                continue;
                                        case FLOATING:
@@ -1975,7 +1978,13 @@ eval(Expr_t* ex, Exnode_t* expr, void* env)
                        return tmp.data.constant.value;
                case EQ:
                case NE:
-                       v.integer = ((v.string && r.string) ? ((ex->disc->version >= 19981111L && ex->disc->matchf) ? (*ex->disc->matchf)(ex, expr->data.operand.left, v.string, expr->data.operand.right, r.string, env, ex->disc) : strmatch(v.string, r.string)) : (v.string == r.string)) == (expr->op == EQ);
+                       v.integer = ((v.string && r.string)
+                                     ? ((ex->disc->version >= 19981111L && ex->disc->matchf)
+                                       ? ex->disc->matchf(ex, expr->data.operand.left, v.string,
+                                                          expr->data.operand.right, r.string, env,
+                                                          ex->disc)
+                                       : strmatch(v.string, r.string))
+                                     : (v.string == r.string)) == (expr->op == EQ);
                        return v;
                case '+':
                        v.string = str_add(ex, v.string, r.string);