From: Regina Obe <lr@pcorp.us>
Date: Thu, 8 Aug 2013 21:19:41 +0000 (+0000)
Subject: #2396: liblwgeom CUnit tests fail on big-endian architectures.
X-Git-Tag: 2.2.0rc1~1419
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a049919b1b8c982292fb237248ddb3b60ee19854;p=postgis

#2396: liblwgeom CUnit tests fail on big-endian architectures.

git-svn-id: http://svn.osgeo.org/postgis/trunk@11749 b70326c6-7e19-0410-871a-916f4a2858ee
---

diff --git a/liblwgeom/cunit/cu_libgeom.c b/liblwgeom/cunit/cu_libgeom.c
index 44d7cbffd..ae0e319f5 100644
--- a/liblwgeom/cunit/cu_libgeom.c
+++ b/liblwgeom/cunit/cu_libgeom.c
@@ -694,6 +694,7 @@ static void test_lwgeom_clone(void)
 static void test_lwgeom_force_clockwise(void)
 {
 	LWGEOM *geom;
+	LWGEOM *geom2;
 	char *in_ewkt, *out_ewkt;
 
 	/* counterclockwise, must be reversed */
@@ -744,13 +745,14 @@ static void test_lwgeom_force_clockwise(void)
 	/* NOTE: this is a narrow ring, see ticket #1302 */
 	in_ewkt  = "0103000000010000000500000000917E9BA468294100917E9B8AEA2841C976BE1FA4682941C976BE9F8AEA2841B39ABE1FA46829415ACCC29F8AEA284137894120A4682941C976BE9F8AEA284100917E9BA468294100917E9B8AEA2841";
 	geom = lwgeom_from_hexwkb(in_ewkt, LW_PARSER_CHECK_NONE);
-	lwgeom_force_clockwise(geom);
-	out_ewkt = lwgeom_to_hexwkb(geom, WKB_ISO, NULL);
-	if (strcmp(in_ewkt, out_ewkt))
-		fprintf(stderr, "\nExp:   %s\nObt:   %s\n", in_ewkt, out_ewkt);
-	CU_ASSERT_STRING_EQUAL(in_ewkt, out_ewkt);
-	lwfree(out_ewkt);
+	geom2 = lwgeom_from_hexwkb(in_ewkt, LW_PARSER_CHECK_NONE);
+	lwgeom_force_clockwise(geom2);
+	
+	/** use same check instead of strcmp to account 
+	  for difference in endianness **/
+	CU_ASSERT( lwgeom_same(geom, geom2) );
 	lwgeom_free(geom);
+	lwgeom_free(geom2);
 }
 
 /*