Fix documentation orient2d

README.md

@@ -31,7 +31,7 @@ The library is published on Maven Central. For Maven, you can include the follow
 <dependency>
     <groupId>org.geolatte</groupId>
     <artifactId>geolatte-geom</artifactId>
-    <version>1.8.0</version>
+    <version>1.9.0</version>
 </dependency>
 ```
 

geom/src/main/java/org/geolatte/geom/GeometryPositionEquality.java

@@ -59,7 +59,8 @@ public GeometryPositionEquality() {
         this.pointSeqEq = new PositionSequencePositionEquality(pointEquality);
     }
 
-
+    //FIXME - provide a means to also inject an PositionSequenceEquality, e.g.
+    // for rings equal except one is shifted n positions w.r.t the other.
 
 
     /**

geom/src/main/java/org/geolatte/geom/cga/NumericalMethods.java

@@ -128,7 +128,7 @@ public static boolean isCounterClockwise(LinearRing<?> ring) {
      * This uses Newell's method (see Graphics Gems III, V. 5)
      *
      * @param ring the linear ring
-     * @return +1 if the specified ring is clockwise, -1 if counterclockwise, 0 if all vertices are collinear
+     * @return -1 if the specified ring is clockwise, +1 if counterclockwise, 0 if all vertices are collinear
      */
     public static double orient2d(LinearRing<?> ring) {
         return orient2d(ring.getPositions());

geom/src/test/java/org/geolatte/geom/cga/NumericalMethodsTest.java

@@ -127,11 +127,15 @@ public void testColinearZigZag() {
 
     @Test
     public void testCounterClockwiseLineString() {
-        LinearRing<C2D> cwLinestring  = ring(xy, c(0, 0), c(1, 0), c(2, 0.5), c(1, -1),   c(0, 0));
-        LinearRing<C2D> ccwLinestring = ring(xy, c(0, 0), c(1, 0), c(2, 0.5), c(1, 0.5), c(0, 0));
+        LinearRing<C2D> cwLinestring  = ring(xy, c(0, 0), c(0, 1), c(1, 1), c(1, 0),   c(0, 0));
+        LinearRing<C2D> ccwLinestring = ring(xy, c(0, 0), c(1, 0), c(1, 1), c(0, 1), c(0, 0));
 
         assertFalse(NumericalMethods.isCounterClockwise(cwLinestring));
+        assertEquals((int)orient2d(cwLinestring), -1);
+
         assertTrue(NumericalMethods.isCounterClockwise(ccwLinestring));
+        assertEquals((int)orient2d(ccwLinestring), 1);
+
     }
 
     @Test