]> granicus.if.org Git - graphviz/commit
cgraph: fix mismatched type signatures between 'agobjfn_t' and 'agraphattr_init'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 6 Nov 2022 18:19:29 +0000 (10:19 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 10 Nov 2022 16:47:34 +0000 (08:47 -0800)
commitf0d28f7609cb4fea6a7ee5393958642aa65b545d
treeb93a6a508ea65ffeea0cfc3534e2a9edec71df25
parentf48e7e4d9692131c8f87023fa9ab0fd31bef59b1
cgraph: fix mismatched type signatures between 'agobjfn_t' and 'agraphattr_init'

Squashes a compiler warning:

  attr.c: In function ‘init_all_attrs’:
  attr.c:534:37: warning: cast between incompatible function types from ‘void
    (*)(Agraph_t *)’ {aka ‘void (*)(struct Agraph_s *)’} to ‘void (*)(Agraph_t
    *, Agobj_t *, void *)’ {aka ‘void (*)(struct Agraph_s *, struct Agobj_s *,
    void *)’} [-Wcast-function-type]
    534 |     agapply(root, (Agobj_t *) root, (agobjfn_t) agraphattr_init,
        |                                     ^

This code was incorrect. But on the most common architecture in contemporary
use, x86-64, the calling convention for both is fully in registers, so no user
visible misbehavior would have been seen.

Gitlab: #2300
lib/cgraph/attr.c