From: Paul Ramsey Date: Tue, 2 Jul 2013 22:33:15 +0000 (+0000) Subject: #2307, test and comment on free line X-Git-Tag: 2.2.0rc1~1461 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=42b77c0b60078f8743f9185c2aab6f282676e10f;p=postgis #2307, test and comment on free line git-svn-id: http://svn.osgeo.org/postgis/trunk@11620 b70326c6-7e19-0410-871a-916f4a2858ee --- diff --git a/liblwgeom/lwgeom_geos_clean.c b/liblwgeom/lwgeom_geos_clean.c index 5c13204b1..52a0c7e94 100644 --- a/liblwgeom/lwgeom_geos_clean.c +++ b/liblwgeom/lwgeom_geos_clean.c @@ -1053,7 +1053,8 @@ lwgeom_make_valid(LWGEOM* lwgeom_in) { LWDEBUG(3, "lwgeom_make_valid: forcing multi"); lwgeom_tmp = lwgeom_as_multi(lwgeom_out); - // lwgeom_free(lwgeom_out); + /* Don't free lwgeom_out, or you'll free memory underneath lwgeom_tmp */ + /* lwgeom_free(lwgeom_out); */ lwgeom_out = lwgeom_tmp; } diff --git a/regress/tickets.sql b/regress/tickets.sql index 21179c72f..e8d5d40a4 100644 --- a/regress/tickets.sql +++ b/regress/tickets.sql @@ -819,6 +819,8 @@ SELECT '#2110.3', 'POINT(0 0)'::geometry = 'POINT(0 0)'::geometry; SELECT '#2145', round(ST_Length(St_Segmentize(ST_GeographyFromText('LINESTRING(-89.3000030518 28.2000007629,-89.1999969482 89.1999969482,-89.1999969482 89.1999969482)'), 10000))::numeric,0); +-- #2307 -- +SELECT '#2307', ST_AsText(ST_SnapToGrid(ST_MakeValid('0106000020E6100000010000000103000000010000000 A0000004B7DA956B99844C0DB0790FE8B4D1DC010BA74A9AF9444C049AFFC5B8C4D1DC03FC6CC6 90D9844C0DD67E5628C4D1DC07117B56B0D9844C0C80ABA67C45E1DC0839166ABAF9444C0387D4 568C45E1DC010BA74A9AF9444C049AFFC5B8C4D1DC040C3CD74169444C0362EC0608C4D1DC07C1 A3B77169444C0DC3ADB40B2641DC03AAE5F68B99844C0242948DEB1641DC04B7DA956B99844C0D B0790FE8B4D1DC0'::geometry),0.0001)); -- Clean up DELETE FROM spatial_ref_sys; diff --git a/regress/tickets_expected b/regress/tickets_expected index d2579fcd2..85f763239 100644 --- a/regress/tickets_expected +++ b/regress/tickets_expected @@ -243,3 +243,4 @@ ERROR: invalid GML representation #2110.2|t #2110.3|t #2145|6792004 +#2307|MULTIPOLYGON(((-41.1932 -7.3257,-41.1616 -7.3257,-41.1569 -7.3257,-41.1569 -7.3483,-41.1932 -7.3483,-41.1932 -7.3257),(-41.1616 -7.3257,-41.1879 -7.3257,-41.1879 -7.3425,-41.1616 -7.3425,-41.1616 -7.3257)))