Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#25876 closed Cleanup/optimization (fixed)

Remove unused/untested OGRGeometry.transform_to() method

Reported by: Sergey Fedoseev Owned by: Sergey Fedoseev
Component: GIS Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

I propose to deprecate OGRGeometry.transform_to() method because it's just an alias for OGRGeometry.transform() method, it's docstring said "For backwards-compatibility." from it's first appearance in the code.

Change History (7)

comment:1 by Sergey Fedoseev, 8 years ago

Owner: changed from nobody to Sergey Fedoseev
Status: new → assigned

comment:2 by Tim Graham, 8 years ago

Triage Stage: Unreviewed → Accepted

It seems undocumented and untested -- maybe it's fine to remove without a deprecation?

comment:3 by Sergey Fedoseev, 8 years ago

This seems to be fine for me.

comment:4 by Sergey Fedoseev, 8 years ago

Has patch: set

PR -- ​https://github.com/django/django/pull/5787

comment:5 by Claude Paroz, 8 years ago

Triage Stage: Accepted → Ready for checkin

comment:6 by Sergey Fedoseev <fedoseev.sergey@…>, 8 years ago

Resolution: → fixed
Status: assigned → closed

In a8614fb:

Fixed #25876 -- Removed OGRGeometry.transform_to() backwards-compatibility method.

comment:7 by Tim Graham, 8 years ago

Summary: deprecate OGRGeometry.transform_to() method → Remove unused/untested OGRGeometry.transform_to() method
Note: See TracTickets for help on using tickets.
Back to Top