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