]> granicus.if.org Git - graphviz/commit
fix: consistently treat entry indices as uint64_t in tclhandle
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Thu, 22 Apr 2021 02:10:19 +0000 (19:10 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 28 Apr 2021 14:54:20 +0000 (07:54 -0700)
commitfafbc9b66f81069a048265f4e9a0a76ca7195c70
tree6cc12809b486d0d4263d073dbb9d4f9cf8db41bc
parent978ceace8a9ce0aefc5f09008e959630c81461a5
fix: consistently treat entry indices as uint64_t in tclhandle

This code intermingled uint64_t and int values for representing indices into an
array. Making this consistent removes a huge number of -Wsign-conversion and
-Wsign-compare compiler warnings, as well as more serious problem where printing
code assumed "%lu" was the format code to print an index. This is not true when
uint64_t is not the same type as unsigned long and would have most probably
resulted in stack corruption.
CHANGELOG.md
tclpkg/tclhandle/tclhandle.c
tclpkg/tclhandle/tclhandle.h
tclpkg/tclpathplan/tclpathplan.c