Opened 18 years ago

Last modified 15 years ago

#97 closed defect (fixed)

SRID value is thrown away by WKBReader

Reported by: cfis@… Owned by: strk@…
Priority: major Milestone:
Component: Core Version: 2.2.2
Severity: Significant Keywords: imported, phpbugtracker
Cc: mloskot

Description (last modified by mloskot)

The WKBReader will read an SRID value, but then throws it away. The attached patch backports a fix from GEOS 3.0 Head. I've been using the patch for several months without problems, and there are tests included in the Ruby test suite.

Attachments (3)

cfis_srid.patch​ (2.0 KB ) - added by cfis@… 18 years ago.
Patch
cfis_srid1.patch​ (2.0 KB ) - added by cfis@… 18 years ago.
Replaces previous patch which had an error.
cfis_srid2.patch​ (2.0 KB ) - added by cfis@… 18 years ago.
Replaces previous patch

Download all attachments as: .zip

Change History (10)

by cfis@…, 18 years ago

Attachment: cfis_srid.patch​ added

Patch

by cfis@…, 18 years ago

Attachment: cfis_srid1.patch​ added

Replaces previous patch which had an error.

by cfis@…, 18 years ago

Attachment: cfis_srid2.patch​ added

Replaces previous patch

comment:1 by strk@…, 18 years ago

Resolution: none → fixed

comment:2 by sgillies@…, 18 years ago 

Howard and I noticed problems with this patch. I've restored the break statements missing from the switch in r1.11.2.3. Tests are all passing, and everything seems fine now.

comment:3 by cfis@…, 18 years ago 

Right, stupid mistake.  Not sure how that go through the tests I had, I'll go look.

comment:4 by mloskot, 17 years ago

Description: modified (diff)
Milestone: imported → 3.0.0

comment:5 by mloskot, 17 years ago

Cc: mloskot added

comment:6 by mloskot, 17 years ago

Priority: 3 → major

comment:7 by (none), 15 years ago

Milestone: 3.0.0

Milestone 3.0.0 deleted

Note: See TracTickets for help on using tickets.