Opened 8 years ago

Closed 8 years ago

#28558 closed Cleanup/optimization (fixed)

Simplify code by removing OGRIndexError

Reported by: Nick Pope Owned by: Nick Pope
Component: GIS Version: dev
Severity: Normal Keywords: gis
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Ticket #25225 cleaned up GEOSIndexError as template engine lookups began handling IndexError and it was no longer required. (comment)

It looks as though this fix for catching IndexError was made in #23489.

The proposal is to make the equivalent changes as in #25225 for OGRIndexError which also allows for related simplifications, e.g. removal of __iter__ in many GDAL objects.

Change History (4)

comment:1 by Nick Pope, 8 years ago

Has patch: set

​PR

comment:2 by Tim Graham, 8 years ago

Patch needs improvement: set
Triage Stage: Unreviewed → Accepted

comment:3 by Nick Pope, 8 years ago

Patch needs improvement: unset

comment:4 by Tim Graham <timograham@…>, 8 years ago

Resolution: → fixed
Status: assigned → closed

In 9397d3ad:

Fixed #28558 -- Simplified code to remove OGRIndexError.

The test is a regression for refs #4740 to show that the original fix of
OGRIndexError is no longer needed.

This is similar to the removal of GEOSIndexError in
197b1878105504b5ac7e399e1f52a6093c88baa3.

Note: See TracTickets for help on using tickets.
Back to Top