Opened 13 years ago

Closed 13 years ago

#396 closed defect (fixed)

missing Covers() and CoveredBy() in CAPI

Reported by: esseffe Owned by: pramsey
Priority: major Milestone:
Component: C API Version: main
Severity: Unassigned Keywords: CoveredBy, history
Cc:

Description

I casually noticed that GEOSCovers() and GEOSCoveredBy() aren't supported at all by the C-API; I suppose this is simply due to a quite trivial omission.

I've then patched both GEOS 3.2.2 and GEOS-trunk, simply cloning the GEOSContains() code so to implement GEOSCovers() and GEOSCoveredBy(); and anything runs in the most pleasant way.

Please see the attached patches.

bye
Sandro Furieri
[developer/maintainer of SpatiaLite]

Attachments (2)

geos_capi_patches.zip​ (1.2 KB ) - added by esseffe 13 years ago.
CAPI patches; GEOSCovers() and GEOSCoveredBy()
capi_covered_by.patch​ (3.7 KB ) - added by esseffe 13 years ago.
svn diff [trunk]

Download all attachments as: .zip

Change History (4)

by esseffe, 13 years ago

Attachment: geos_capi_patches.zip​ added

CAPI patches; GEOSCovers() and GEOSCoveredBy()

comment:1 by strk, 13 years ago

Could you please attach a single patch file not in .zip format and applying against trunk ? Output of an svn diff would be good. Thanks

by esseffe, 13 years ago

Attachment: capi_covered_by.patch​ added

svn diff [trunk]

comment:2 by strk, 13 years ago

Keywords: CoveredBy history added; CovereBy removed
Resolution: → fixed
Status: new → closed

r3249 brings this in.

Note: See TracTickets for help on using tickets.