]> granicus.if.org Git - graphviz/commit
scformat: use vmalloc instead of vmresize when processing '['
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Mon, 19 Jul 2021 00:09:59 +0000 (17:09 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Wed, 28 Jul 2021 15:02:00 +0000 (08:02 -0700)
commit60d4322f57b92f20089d494764fbc9f26eb49b04
tree3ee4cbce31f01fed570f4c8d5f8a5300093260aa
parentec699de44371d384d59164faf80a990ae6f3ded5
scformat: use vmalloc instead of vmresize when processing '['

Though it is hard to see, the previous code bottomed out in a call to vmresize.
After allocating, it zeroes the allocated region. So vmresize (itself calling
realloc) signals an unnecessary constraint to the allocator that previous data
in this memory must be preserved.

To simplify this, this change converts the sequence to a vmfree of the prior
memory and then a new vmalloc, more clearly divorcing the new buffer from the
old one.
lib/expr/exeval.c