]> granicus.if.org Git - graphviz/commit
sfdpgen check_real_array_size: abbreviate no-op 'MAX' call
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 28 Aug 2022 20:11:26 +0000 (13:11 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 31 Aug 2022 00:12:49 +0000 (17:12 -0700)
commit1c5e605e590409a77a835db1d57bd3b669442ebe
treedb2fcf985b37ba38964995f8e9c39f636d8bd312
parentf2200740c4ac71f405f0f1f601b7dbccfafb8097
sfdpgen check_real_array_size: abbreviate no-op 'MAX' call

1b847b5abf50d2ab0701523b90a20306d0ee5528 seems to have incorrectly assumed
multiplication has a higher precedence than casts. In reality, it is the
opposite, meaning the first parameter to this `MAX` call was always 0. So the
entire expression would evaluate to 10. Empirically this seems to have been fine
since this code has been in use for over a decade with no specific problems
blamed on this area. So lets just abbreviate it into what it evaluates to.

Gitlab: #2269
lib/sfdpgen/spring_electrical.c