]> granicus.if.org Git - graphviz/commitdiff
expr: remove further unused parameters from 'Exdisc_t.reff'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 5 Aug 2022 03:49:08 +0000 (20:49 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Fri, 5 Aug 2022 03:55:25 +0000 (20:55 -0700)
I guess when I committed ef3ed2a98a46d0641d936c62fde78e719cef98f8 it was not
obvious to me that this is entirely part of private internal interfaces and the
callback signature involved is not exposed publicly.

lib/expr/exgram.h
lib/expr/exparse.y
lib/expr/expr.h
lib/expr/extoken.c
lib/gvpr/compile.c

index 85e58ab094af5aa5da03b09d791852c02959f214..cf5dad23b50cef7619f4093bceeed281f35cd015 100644 (file)
@@ -398,8 +398,7 @@ static Exnode_t *makeVar(Expr_t * prog, Exid_t * s, Exnode_t * idx,
            exerror("%s: identifier references not supported", sym->name);
        else if (expr.program->disc->reff)
            (*expr.program->disc->reff) (prog, nn,
-                                        nn->data.variable.symbol, refs,
-                                        NULL, EX_SCALAR);
+                                        nn->data.variable.symbol, refs);
 
        return nn;
 }
index 09a9126ab35ef825208cf4a9439d15ef14df26f4..1d86eeda72cbb81163b12b4678d23ac4de133531 100644 (file)
@@ -1009,7 +1009,7 @@ constant  :       CONSTANT
                        if (!expr.program->disc->reff)
                                exerror("%s: identifier references not supported", $1->name);
                        else
-                               $$->data.constant.value = (*expr.program->disc->reff)(expr.program, $$, $1, NULL, NULL, EX_SCALAR);
+                               $$->data.constant.value = (*expr.program->disc->reff)(expr.program, $$, $1, NULL);
                }
                |       FLOATING
                {
index 933299c204984e9b2cd068756ea0a04b169d80f2..fe6aa21c94fc6e56b80588a99c9df393a5e5acbc 100644 (file)
@@ -215,7 +215,7 @@ struct Exdisc_s                             /* discipline                   */
        Exerror_f       errorf;         /* error function               */
        Extype_t        (*getf)(Expr_t*, Exnode_t*, Exid_t*, Exref_t*, void*, int, Exdisc_t*);
                                        /* get value function           */
-       Extype_t        (*reff)(Expr_t*, Exnode_t*, Exid_t*, Exref_t*, char*, int);
+       Extype_t        (*reff)(Expr_t*, Exnode_t*, Exid_t*, Exref_t*);
                                        /* reference value function     */
        int             (*setf)(Expr_t*, Exnode_t*, Exid_t*, Exref_t*, void*, int, Extype_t, Exdisc_t*);
                                        /* set value function           */
index 8864b1e2491fad237cded93785bc2d17ea05004c..b736d5252f0aa1738a1639a1888889588af4bea2 100644 (file)
@@ -859,7 +859,7 @@ extoken_fn(Expr_t* ex)
                        }
                        break;
                }
-               (*ex->disc->reff)(ex, NULL, ex_lval.id, NULL, exstash(ex->tmp, NULL), 0);
+               (*ex->disc->reff)(ex, NULL, ex_lval.id, NULL);
 
                                                /*..INDENT*/
                                        }
index 303bc34d4845cbecfcf81f839bcaa1612e1d9b95..2f3fdca41ba4567ef9a9c23ac11879f6ecca14a1 100644 (file)
@@ -1808,11 +1808,8 @@ static tctype typeChkExp(Exref_t * ref, Exid_t * sym)
  * Type check expressions; return value unused.
  */
 static Extype_t
-refval(Expr_t * pgm, Exnode_t * node, Exid_t * sym, Exref_t * ref,
-       char *str, int elt)
+refval(Expr_t * pgm, Exnode_t * node, Exid_t * sym, Exref_t * ref)
 {
-    (void)str;
-    (void)elt;
 
     Extype_t v;
     if (sym->lex == CONSTANT) {