ST_RemoveRepeatedPoints can change endpoint

[strk]

I don't think ST_RemoveRepeatedPoints should move endpoints. The last point can instead disappear:

=$ select ST_AsText(ST_RemoveRepeatedPoints('LINESTRING(10 0,10 9,10 10)', 2));
       st_astext       
-----------------------
 LINESTRING(10 0,10 9)
(1 row)

[strk]

Ok, this is debatable, maybe better discussed on mailing list

[strk]

Indeed the last point is retained IFF the line is a 2 vertices line:

select ST_AsText(ST_RemoveRepeatedPoints('LINESTRING(10 0,10 1)', 2));
       st_astext       
-----------------------
 LINESTRING(10 0,10 1)
(1 row)

[strk]

This is even worst on polygon, where the function returns invalid ones. Paul, I think this is important to fix.

[pramsey]

agreed, I think start/end points are pretty sacrosant

[strk]

Paul, are you going to fix this one then ?

[pramsey]

Yes

[pramsey]

Fixed in trunk at r14493

[pramsey]

Fixed in 2.2 at r14494