Opened 2 years ago

Closed 2 years ago

#28558 closed Cleanup/optimization (fixed)

Simplify code by removing OGRIndexError

Reported by: Nick Pope Owned by: Nick Pope
Component: GIS Version: master
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 Changed 2 years ago by Nick Pope

Has patch: set

comment:2 Changed 2 years ago by Tim Graham

Patch needs improvement: set
Triage Stage: UnreviewedAccepted

comment:3 Changed 2 years ago by Nick Pope

Patch needs improvement: unset

comment:4 Changed 2 years ago by Tim Graham <timograham@…>

Resolution: fixed
Status: assignedclosed

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