Opened 9 years ago
Closed 8 years ago
#25938 closed Cleanup/optimization (fixed)
deduplicate GEOSBase and GDALBase
Reported by: | Sergey Fedoseev | Owned by: | Sergey Fedoseev |
---|---|---|---|
Component: | GIS | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
gis.gdal.base.GDALBase
and gis.geos.base.GEOSBase
classes have almost the same implementation, their subclasses uses the same pattern for __del__()
method, so all this logic should be factored out to a common class.
P.S.: suggestions for the name of this new common class are welcome.
Change History (8)
comment:1 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:2 by , 9 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:5 by , 9 years ago
Triage Stage: | Accepted → Ready for checkin |
---|
comment:6 by , 9 years ago
Patch needs improvement: | set |
---|---|
Triage Stage: | Ready for checkin → Accepted |
Note:
See TracTickets
for help on using tickets.
Maybe a name like
CPointerBase
?