]> granicus.if.org Git - graphviz/commit
fix GVPR incorrect interpretation of color strings
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 30 Sep 2021 03:29:01 +0000 (20:29 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 2 Oct 2021 01:14:21 +0000 (18:14 -0700)
commitec384a3408022c4d5f35f19d0b540dbfb7b061dd
treed54b8fc9cb0c80ef3d4603f35a6a63b9a4960370
parenta38a79732639e849bf17674811227e030ccbc305
fix GVPR incorrect interpretation of color strings

The logical operators here were pretty clearly meant to be `&&` not `||`. But
this kind of micro-optimization is unnecessary on modern machines anyway, so we
can just rewrite this into something that is more obvious for both readers and
the compiler.

It would have been nice to add a provoking test case for this as well, but it is
not clear to me how exactly to reach this path. This bug has been open for quite
a while, so it seems simpler to apply the obvious fix and move on.

Fixes #1956.
CHANGELOG.md
lib/gvpr/actions.c