Opened 14 years ago
Closed 14 years ago
#831 closed enhancement (fixed)
Flags for ST_IsValid*
Reported by: | strk | Owned by: | strk |
---|---|---|---|
Priority: | high | Milestone: | PostGIS 2.0.0 |
Component: | postgis | Version: | master |
Keywords: | Cc: |
Description
Allow ST_IsValid (and friends) to take flags driving run. In particular, we'll want to allow exposing the SelfTouchingRingFormingHoleValid flag.
Change History (3)
comment:1 by , 14 years ago
comment:2 by , 14 years ago
I've resolved to add 2 signatures for ST_isValid(geom,int) and ST_isValidReason(geom,int) which are _only_ given when building againts GEOS-3.3 or up and are implemented as simple wrappers to ST_isValidDetail.
Will commit with documentation and testing.
Note:
See TracTickets
for help on using tickets.
As of r6826 the flag is allowed in ST_IsValidDetail. Since that function is already new, and requiring GEOS 3.3.0 I tought it'd be useful to have the flag there.
Now, if we also want to have support in the other two (ST_isValid and ST_isValidReason) we'll need to switch implementation based on GEOS version, with the pre-geos-3.3 implementation raising an exception if a flag != 0 is given.
Do you think it's worth it ?