]> granicus.if.org Git - graphviz/commit
remove use of double literals with ydelta in make_flat_labeled_edge
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 2 May 2021 03:03:51 +0000 (20:03 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 16 May 2021 16:40:28 +0000 (09:40 -0700)
commitdb09ae06fddad8f8153e64c5a1163180a2005707
treea087a20f2f5f466d7542ea60615a6df33aafdd29
parent363f9bc05380cfc500fe94261e786b0c5e6adc67
remove use of double literals with ydelta in make_flat_labeled_edge

Commit ebe7517ae4eb86ee4bf34e1af31a174f1f0a4049 altered these lines to use
double literals instead of integer literals for `5` and `6`. It is not clear why
it did this as ydelta is an integer variable. The effect of that change was to
ask the compiler to perform the computation itself using doubles and then reduce
it to an integer result. This commit reverts that part of the diff to simply use
integer computation throughout. This change was motivated by the compiler
diagnosing this issue with -Wfloat-conversion.
lib/dotgen/dotsplines.c