]> granicus.if.org Git - graphviz/commit
remove cgraph macro aliases
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 26 Dec 2021 03:56:19 +0000 (19:56 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Tue, 4 Jan 2022 02:05:43 +0000 (18:05 -0800)
commitb7e9e1d9e9e522961cc3f0ab316852c6649ef7a2
tree7580093da783b5a56a09514b921e581b31181959
parent0f89b538b45c2c028fb8c8abe3c08914f16ae92f
remove cgraph macro aliases

The functions removed in this commit were intended to be aliases for the
same-named macros declared in cgraph.h. The claimed purpose is debugger
introspection.

It is unclear to me what value these have in a modern development environment.
Some modern debuggers can understand macro definitions. Those that cannot are
_impeded_ by these aliases. A developer debugging looks at the alias definition
and thinks they understand the code, but they do not. The code that is executing
on their system is _not_ the code these functions compile to. To state something
stronger: it is unclear to me how these definitions were ever helpful. They
appear to (accidentally) obscure what is actually occurring in the machine at
runtime.

Note that these functions also bloated the cgraph .so/.a size by including
functions that were never called.
lib/cgraph/edge.c