[Gdal-dev] [Patch] Problem with gdal_translate with the parameter -a_ullr 0 0 x x

[Mateusz Loskot]

Jean-Claude Repetto wrote:
> Jean-Claude Repetto wrote :
>>
>> The ECW file is correct. I have checked it by reading it directly with
>> the ECW library.
>> The bug occurs when GDAL reads the file. I have looked at the source
>> code (version 1.4.2), and it seems the bug is in the file
>> ecwdataset.cpp .
> 
> This is a patch that I propose :
> 
> --- frmts/ecw/ecwdataset.cpp.orig       2007-10-24 11:35:44.000000000 +0200
> +++ frmts/ecw/ecwdataset.cpp    2007-10-24 11:41:59.000000000 +0200
> @@ -1539,9 +1539,9 @@
> 
>      if( psFileInfo->fOriginX != 0.0
>          || psFileInfo->fOriginY != 0.0
> -        && (psFileInfo->fCellIncrementX != 0.0 &&
> +        || (psFileInfo->fCellIncrementX != 0.0 &&
>              psFileInfo->fCellIncrementX != 1.0)
> -        && (psFileInfo->fCellIncrementY != 0.0 &&
> +        || (psFileInfo->fCellIncrementY != 0.0 &&
>              psFileInfo->fCellIncrementY != 1.0) )
>      {
>          bGeoTransformValid = TRUE;
> 
> 
> With this patch applied to GDAL 1.4.2, both gdalinfo and qgis give
> correct results.

Jean-Claude,

Please, submit this patch as a ticket. We will test it. Thanks!


Cheers
-- 
Mateusz Loskot
http://mateusz.loskot.net