Opened 12 years ago
Closed 12 years ago
#4341 closed defect (fixed)
Rasdaman should not implement GetGeoTransform()
| Reported by: | fschindler | Owned by: | warmerdam |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.9.0 |
| Component: | GDAL_Raster | Version: | 1.8.1 |
| Severity: | normal | Keywords: | rasdaman |
| Cc: | schpidi, jucovschi@… |
Description
Rasdaman datasets do not support georeferencing, thus the method GetGeoTransform should return CE_Failure (as in other formats that do not support georeferencing), but currently CE_None is returned.
Attachments (1)
Change History (2)
by , 12 years ago
| Attachment: | gdal_rasdaman_getgeotransform.patch added |
|---|
comment:1 by , 12 years ago
| Cc: | added |
|---|---|
| Component: | default → GDAL_Raster |
| Milestone: | → 1.9.0 |
| Resolution: | → fixed |
| Status: | new → closed |
| Summary: | Found bug in rasdaman dataset → Rasdaman should not implement GetGeoTransform() |
Looks like this ticket is closely tied with a thread ("MapServer integration problem" : http://groups.google.com/group/rasdaman-dev/browse_thread/thread/3bc20545f9da989c#) on rasdaman-dev mailing list. I find it good when ticket reporters can add more contexts to their tickets. If I hadn't been subscribed to rasdaman-dev, the impact of this ticket would have not been very obvious.
I've commited a different fix that should have identical result. I've just removed the RasdamanDataset::GetGeoTransform() implementation which, with the proposed patch, is identical to the GDALDataset::GetGeoTransform() base implementation.
r23362 /trunk/gdal/frmts/rasdaman/rasdamandataset.cpp: Rasdaman: remove implementation of GetGeoTransform() because Rasdaman driver does not support georeferencing (#4341)
Patch resolving the issue