#242 closed defect (fixed)
GEOSSetSRID_r missing, GEOSSetSRID duplicated
| Reported by: | tazle | Owned by: | mloskot |
|---|---|---|---|
| Priority: | major | Milestone: | 3.2.0 |
| Component: | C API | Version: | 3.1.0 |
| Severity: | Unassigned | Keywords: | |
| Cc: |
Description (last modified by )
Currently there is no GEOSSetSRID_r declaration or implementation in the C API. Instead there is another GEOSSetSRID implementation in the thread-safe implementation and two GEOSSetSRID (second one) declarations in the header.
I think there should be GEOSSetSRID_r for consistency even if GEOSSetSRID is thread-safe (it seems like it might be) - certainly all other functions have _r counterparts.
Attachments (1)
Change History (7)
comment:1 by , 15 years ago
| Description: | modified (diff) |
|---|---|
| Milestone: | 3.1.1 |
comment:2 by , 15 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:3 by , 15 years ago
| Milestone: | → 3.2.0 |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
comment:4 by , 15 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Unfortunately, this submission touches public API and we are not working on a dedicated branch (yet). So, I've reverted (r2301) and preparing a file-based patch.
by , 15 years ago
| Attachment: | geossetsrid-ts-ticket-242.patch added |
|---|
Patch with implementation of missing GEOSSetSRID_r. It's going to be applied to trunk after GEOS 3.1.0 is released.
comment:5 by , 15 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Committed as revision 2491.
Note:
See TracTickets
for help on using tickets.
Fixed in trunk (r2300). Please, test and confirm it works, reopen the ticket if it doesn't.