From d0b775263c7cec8eebc9320e3da1ee5028d0e1b0 Mon Sep 17 00:00:00 2001
From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sun, 26 Dec 2021 18:52:23 -0800
Subject: [PATCH] generateNonoverlapConstraints: take a C99 bool instead of
 boolean

Note that this required propagating this change through to `genXConstraints` as
well.
---
 lib/neatogen/constrained_majorization_ipsep.c | 5 +++--
 lib/neatogen/quad_prog_vpsc.c                 | 6 +++---
 lib/neatogen/quad_prog_vpsc.h                 | 3 ++-
 lib/vpsc/csolve_VPSC.cpp                      | 5 +++--
 lib/vpsc/csolve_VPSC.h                        | 5 ++++-
 5 files changed, 15 insertions(+), 9 deletions(-)

diff --git a/lib/neatogen/constrained_majorization_ipsep.c b/lib/neatogen/constrained_majorization_ipsep.c
index e4c9f5288..3c05ee476 100644
--- a/lib/neatogen/constrained_majorization_ipsep.c
+++ b/lib/neatogen/constrained_majorization_ipsep.c
@@ -32,6 +32,7 @@
  **********************************************************/
 
 #include <neatogen/digcola.h>
+#include <stdbool.h>
 #ifdef IPSEPCOLA
 #include <math.h>
 #include <stdlib.h>
@@ -411,7 +412,7 @@ int stress_majorization_cola(vtx_data * graph,	/* Input graph in sparse represen
 	if (opt->noverlap == 1 && nsizeScale > 0.001) {
 	    generateNonoverlapConstraints(cMajEnvHor, nsizeScale, coords,
 					  0,
-					  nsizeScale < 0.5 ? FALSE : TRUE,
+					  nsizeScale >= 0.5,
 					  opt);
 	}
 	if (cMajEnvHor->m > 0) {
@@ -435,7 +436,7 @@ int stress_majorization_cola(vtx_data * graph,	/* Input graph in sparse represen
 	}
 	if (opt->noverlap == 1 && nsizeScale > 0.001) {
 	    generateNonoverlapConstraints(cMajEnvVrt, nsizeScale, coords,
-					  1, FALSE, opt);
+					  1, false, opt);
 	}
 	if (cMajEnvVrt->m > 0) {
 #ifdef MOSEK
diff --git a/lib/neatogen/quad_prog_vpsc.c b/lib/neatogen/quad_prog_vpsc.c
index bdac4fab6..c892e5931 100644
--- a/lib/neatogen/quad_prog_vpsc.c
+++ b/lib/neatogen/quad_prog_vpsc.c
@@ -375,7 +375,7 @@ void generateNonoverlapConstraints(CMajEnvVPSC * e,
 				   float nsizeScale,
 				   float **coords,
 				   int k,
-				   boolean transitiveClosure,
+				   bool transitiveClosure,
 				   ipsep_options * opt)
 {
     Constraint **csol, **csolptr;
@@ -587,12 +587,12 @@ void removeoverlaps(int n, float **coords, ipsep_options * opt)
 {
     int i;
     CMajEnvVPSC *e = initCMajVPSC(n, NULL, NULL, opt, 0);
-    generateNonoverlapConstraints(e, 1.0, coords, 0, TRUE, opt);
+    generateNonoverlapConstraints(e, 1.0, coords, 0, true, opt);
     solveVPSC(e->vpsc);
     for (i = 0; i < n; i++) {
 	coords[0][i] = getVariablePos(e->vs[i]);
     }
-    generateNonoverlapConstraints(e, 1.0, coords, 1, FALSE, opt);
+    generateNonoverlapConstraints(e, 1.0, coords, 1, false, opt);
     solveVPSC(e->vpsc);
     for (i = 0; i < n; i++) {
 	coords[1][i] = getVariablePos(e->vs[i]);
diff --git a/lib/neatogen/quad_prog_vpsc.h b/lib/neatogen/quad_prog_vpsc.h
index f9a57372d..7f5b37409 100644
--- a/lib/neatogen/quad_prog_vpsc.h
+++ b/lib/neatogen/quad_prog_vpsc.h
@@ -32,6 +32,7 @@ extern "C" {
 #ifdef MOSEK
 #include <neatogen/mosek_quad_solve.h>
 #endif /* MOSEK */
+#include <stdbool.h>
 
 typedef struct CMajEnvVPSC {
 	float **A;
@@ -64,7 +65,7 @@ extern void generateNonoverlapConstraints(
         float nsizeScale,
         float** coords,
         int k,
-	boolean transitiveClosure,
+	bool transitiveClosure,
 	ipsep_options* opt
 );
 
diff --git a/lib/vpsc/csolve_VPSC.cpp b/lib/vpsc/csolve_VPSC.cpp
index 8002431ff..ef2f004b1 100644
--- a/lib/vpsc/csolve_VPSC.cpp
+++ b/lib/vpsc/csolve_VPSC.cpp
@@ -35,12 +35,13 @@ VPSC* newIncVPSC(int n, Variable* vs[], int m, Constraint* cs[]) {
 	return new IncVPSC(n,vs,m,cs);
 }
 
-int genXConstraints(int n, boxf* bb, Variable** vs, Constraint*** cs,int transitiveClosure) {
+int genXConstraints(int n, boxf *bb, Variable **vs, Constraint ***cs,
+                    bool transitiveClosure) {
 	std::vector<Rectangle> rs;
 	for(int i=0;i<n;i++) {
 		rs.emplace_back(bb[i].LL.x,bb[i].UR.x,bb[i].LL.y,bb[i].UR.y);
 	}
-	int m = generateXConstraints(rs,vs,*cs,transitiveClosure?true:false);
+	int m = generateXConstraints(rs, vs, *cs, transitiveClosure);
 	return m;
 }
 int genYConstraints(int n, boxf* bb, Variable** vs, Constraint*** cs) {
diff --git a/lib/vpsc/csolve_VPSC.h b/lib/vpsc/csolve_VPSC.h
index a5deac895..0ea15ecea 100644
--- a/lib/vpsc/csolve_VPSC.h
+++ b/lib/vpsc/csolve_VPSC.h
@@ -15,6 +15,9 @@
  * Adaptagrams repository.
  */
 #pragma once
+
+#include <stdbool.h>
+
 #ifdef __cplusplus
 extern "C" {
 #endif
@@ -43,7 +46,7 @@ typedef struct pointf_s { double x, y; } pointf;
 typedef struct { pointf LL, UR; } boxf;
 #endif
 int genXConstraints(int n, boxf[], Variable** vs, Constraint*** cs,
-		int transitiveClosure);
+		bool transitiveClosure);
 int genYConstraints(int n, boxf[], Variable** vs, Constraint*** cs);
 
 void satisfyVPSC(VPSC*);
-- 
2.50.1