]> granicus.if.org Git - graphviz/commit
expr: fix misinterpretation of shifts
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 16 Jan 2022 18:47:37 +0000 (10:47 -0800)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 19 Jan 2022 16:21:09 +0000 (08:21 -0800)
commitf1baa04e064631847cb065a0d3af53c51d21e008
treea1bc383625aba8f89976cf5117c169b4a43e50e5
parente422f8722a8b50e917caa02c7c15df32044b8b6c
expr: fix misinterpretation of shifts

It is fairly clear from the surrounding code that these instances were writing
to the wrong destination and, in some cases, reading from the wrong sources. The
effect of this bug was a little slippery as the underlying storage is a union
with `integer` and `floating` occupying the same storage, so this change does
not alter which bytes are stored to. Additionally users rarely left or right
shift within GVPR scripts, so it is unlikely this affected any real world code.

Gitlab: Fixes #2103
CHANGELOG.md
lib/expr/exeval.c