Opened 9 years ago

Closed 8 years ago

Last modified 8 years ago

#210 closed enhancement (fixed)

Create a thread safe CAPI

Reported by: cthibert Owned by: geos-devel@…
Priority: major Milestone: 3.1.0
Component: C API Version: master
Severity: Feature Request Keywords:
Cc:

Description

The current CAPI in GEOS is not thread safe. The error handling and initialization/finalization process specifically can cause problems.

Attachments (4)

Ticket#210.tar.gz (16.8 KB) - added by cthibert 9 years ago.
Includes the new API and test tools. See README.
Ticket#210-1.0.tar.gz (22.0 KB) - added by cthibert 9 years ago.
This is a working version of the API changes. Changes were made following RFC-3. Included is a file 'testgrammar' which will cause errors when parsing and illustrate the threading issue. badthreadtest tries threading with the non-safe api and threadtest uses the new api.
Ticket#210-patch-1.0.gz (24.5 KB) - added by cthibert 8 years ago.
The last tar file attachment does not have all the files required. This patch is taken from the root directory of trunk.
Ticket#210-2.0.tar.gz (21.8 KB) - added by cthibert 8 years ago.
This includes the changes based on Paul's comments. Since the test executables were move I created a script to svn move the files before applying the patch.

Download all attachments as: .zip

Change History (8)

Changed 9 years ago by cthibert

Attachment: Ticket#210.tar.gz added

Includes the new API and test tools. See README.

Changed 9 years ago by cthibert

Attachment: Ticket#210-1.0.tar.gz added

This is a working version of the API changes. Changes were made following RFC-3. Included is a file 'testgrammar' which will cause errors when parsing and illustrate the threading issue. badthreadtest tries threading with the non-safe api and threadtest uses the new api.

Changed 8 years ago by cthibert

Attachment: Ticket#210-patch-1.0.gz added

The last tar file attachment does not have all the files required. This patch is taken from the root directory of trunk.

Changed 8 years ago by cthibert

Attachment: Ticket#210-2.0.tar.gz added

This includes the changes based on Paul's comments. Since the test executables were move I created a script to svn move the files before applying the patch.

comment:1 Changed 8 years ago by pramsey

Milestone: 3.1

comment:2 Changed 8 years ago by pramsey

Resolution: fixed
Status: newclosed

Committed to trunk at r2232.

comment:3 Changed 8 years ago by (none)

Milestone: 3.1.0

Milestone 3.1.0 deleted

comment:4 Changed 8 years ago by pramsey

Milestone: 3.1.0
Version: svn-trunk
Note: See TracTickets for help on using tickets.