From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Sat, 5 Jun 2021 01:18:06 +0000 (-0700)
Subject: fix incorrect return value in inside_polygon
X-Git-Tag: 2.47.3~13^2
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=bdee0834927cbc4a6a940c90da855c08481f8b8e;p=graphviz

fix incorrect return value in inside_polygon

A refactor in 7731b18f403b545ea6eca99f429dd9b63e0659d6 incorrectly flipped the
polarity of one of the return sites in inside_polygon. The effect of this was
confusing assertion failures. Fixes #2082.
---

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0be79cedd..3fbbb96a8 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -22,6 +22,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - out of bounds read when examining a registered plugin whose name is ≥63
   characters
 - do not `fclose(NULL)` in gvmap
+- Assertion error when using `dot` in `ortho.c` in `convertSPtoRoute` in
+  graphviz 2.47.2 #2082. This was a regression introduced in 2.47.2.
 
 ## [2.47.2] - 2021-05-26
 
diff --git a/lib/ortho/partition.c b/lib/ortho/partition.c
index fefce8a04..69b1dbcfd 100644
--- a/lib/ortho/partition.c
+++ b/lib/ortho/partition.c
@@ -181,7 +181,7 @@ inside_polygon (trap_t *t, segment_t* seg)
   if ((t->u0 <= 0 && t->u1 <= 0) || (t->d0 <= 0 && t->d1 <= 0)) /* triangle */
     return _greater_than(&seg[rseg].v1, &seg[rseg].v0);
   
-  return true;
+  return false;
 }
 
 static double
diff --git a/rtest/test_regression.py b/rtest/test_regression.py
index 5dc374d4a..b134e8962 100644
--- a/rtest/test_regression.py
+++ b/rtest/test_regression.py
@@ -816,7 +816,6 @@ def test_2078():
 
   assert "subgraph" in stderr.lower(), "subgraph not mentioned in error message"
 
-@pytest.mark.xfail()
 def test_2082():
   """
   Check a bug in inside_polygon has not been reintroduced.