]> granicus.if.org Git - graphviz/commit
use a local instead of reusing global AF in gvrender_polyline
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sun, 4 Oct 2020 21:00:14 +0000 (14:00 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 10 Oct 2020 21:45:05 +0000 (14:45 -0700)
commitcd80bff8839d4afc1791f1458cad2f5bb01e2f61
treecb4bac608f5d8d2274584cdb3afa29cf38c18fd2
parentf7dfa77d6e8d68d3bdcc1c635546b49ec01ddac4
use a local instead of reusing global AF in gvrender_polyline

There is a static global, AF, that is reused for a number of gvrender functions,
but none need to retain previous data stored in this array. This hack presumably
was from a time when allocators were much slower. Refactoring this into a local
allocation makes this function thread safe and removes the need to unnecessarily
prolong the lifetime of this allocation, thus decreasing Graphviz memory usage.
This commit introduces a -Wshadow warning about AF, but that will be removed
when we soon remove the static global.
lib/gvc/gvrender.c