Opened 7 years ago

Closed 7 years ago

#3780 closed defect (duplicate)

ST_Reverse(curve) has no effect

Reported by: strk Owned by: strk
Priority: medium Milestone: PostGIS 2.4.0
Component: postgis Version: 2.3.x
Keywords: curves Cc:

Description

Check this out:

strk=# select ST_AsText(ST_Reverse('CIRCULARSTRING(0 0, 5 5, 10 0)'::geometry));
          st_astext           
------------------------------
 CIRCULARSTRING(0 0,5 5,10 0)
(1 row)

Change History (6)

comment:1 by strk, 7 years ago

Code to do reversing of circulastrings also exist, just it is not called when it should be:

liblwgeom/lwcircstring.c:131:void lwcircstring_reverse(LWCIRCSTRING *curve)

The lwgeom_reverse function simply doesn't include a CASE for the curve types.

comment:2 by robe, 7 years ago

strk so easy enough to back update to 2.3.4?

comment:3 by strk, 7 years ago

Easy, but I'm not committing to do it, so don't have it hold back anything

comment:4 by robe, 7 years ago

well we probably won't be releasing 2.3.4 for a while given we just released 2.3.3 so you've got plenty of time to think about it.

comment:5 by robe, 7 years ago

Owner: changed from pramsey to strk

comment:6 by strk, 7 years ago

Resolution: duplicate
Status: newclosed

It looks like I already filed this as #2411

Note: See TracTickets for help on using tickets.