From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Fri, 18 Nov 2022 01:14:41 +0000 (-0800)
Subject: sparse get_or_alloc_force_qt: use cgraph wrapper for allocation
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=8baa2bc77bef73caeffb07af1611fabfbb238eee;p=graphviz

sparse get_or_alloc_force_qt: use cgraph wrapper for allocation

The lib/cgraph/alloc.h wrappers are similar to the older lib/common/memory.h
wrappers except (1) they are header-only and (2) they live in a directory
(cgraph) that is at the root of the dependency tree. The long term plan is to
replace all use of lib/common/memory.h with lib/cgraph/alloc.h.
---

diff --git a/lib/sparse/QuadTree.c b/lib/sparse/QuadTree.c
index 4c5367063..4e1692652 100644
--- a/lib/sparse/QuadTree.c
+++ b/lib/sparse/QuadTree.c
@@ -8,6 +8,7 @@
  * Contributors: Details at https://graphviz.org
  *************************************************************************/
 
+#include <cgraph/alloc.h>
 #include <sparse/general.h>
 #include <common/geom.h>
 #include <common/arith.h>
@@ -148,12 +149,10 @@ static double *get_or_assign_node_force(double *force, int i, SingleLinkedList l
   return f;
 }
 static double *get_or_alloc_force_qt(QuadTree qt, int dim){
-  int i;
   double *force = qt->data;
   if (!force){
-    qt->data = MALLOC(sizeof(double)*dim);
+    qt->data = gv_calloc(dim, sizeof(double));
     force = qt->data;
-    for (i = 0; i < dim; i++) force[i] = 0.;
   }
   return force;
 }