Re: [gdal-dev] gdalwarp integer overflow

2013-12-04 Thread Vincent Schut
On 12/03/2013 04:56 PM, Even Rouault wrote: Hi, do you have many bands and what is their data type ? Because the check for integer overflows does nSrcXSize * nSrcYSize * nWordSize * nBandCount. nSrcXSize * nSrcYSize by itself is 214 659 158. So it means that you have more than 10 bands of Byte

Re: [gdal-dev] Setting the noData flag in a GeoTIFF

2013-12-04 Thread Jonathan Moules
Hi Jukka, Good point. In fact, that's one of the reasons I want to set the noData flag. I figure when compressing hopefully the compressor will see that it is noData and will treat it differently from the way it would an actual value (which results in things like your mentioned 255,255,254) so I

Re: [gdal-dev] Setting the noData flag in a GeoTIFF

2013-12-04 Thread Jukka Rahkonen
Jonathan Moules jonathanmoules at warwickshire.gov.uk writes: Hi Jukka,Good point. In fact, that's one of the reasons I want to set the noData flag. I figure when compressing hopefully the compressor will see that it is noData and will treat it differently from the way it would an actual

Re: [gdal-dev] How to get coordinates of original corners from warped image

2013-12-04 Thread Nick Ves
On Sat, Nov 30, 2013 at 1:27 PM, Lauri Kajan lauri.ka...@gmail.com wrote: Hi all I have a set of aerial bird's view images and now I need to get extents of those images as polygons. I thought to add GCPs to images and possibly somehow to use the same transformation that is used when warping

[gdal-dev] VS2010-VS2013 s57

2013-12-04 Thread Dmitriy Baryshnikov
Hi, I have such error: the GDAL compiled with VS2010-VS2013 in s57 driver loose all additional fields values. But the same code compiled with gcc or previous VS works fine. I found the root of problems here (ogr\ogrsf_frmts\s57\s57reader.cpp:932): const char *pszAcronym =

Re: [gdal-dev] VS2010-VS2013 s57

2013-12-04 Thread Frank Warmerdam
Dmitriy, It is not clear to me why this should be necessary. osAcronym is a long lived std::string (well CPLString derived from std::string) living in the registrars attribute list. I assume the following method is used to convert the CPLString to const char * which should amount to the same

Re: [gdal-dev] VS2010-VS2013 s57

2013-12-04 Thread Dmitriy Baryshnikov
Hi Frank, You quite right, what It is not clear to me why this should be necessary. osAcronym is a long lived std::string (well CPLString derived from std::string) living in the registrars attribute list. I assume the following method is used to convert the CPLString to const char

Re: [gdal-dev] VS2010-VS2013 s57

2013-12-04 Thread Even Rouault
to the ticket? Yes, Dmitriy, please commit your change (GetAttrName() would need the same change also). Hopefully, it will solve the failure on ogr_s57_6 that have been seen in http://www.gisinternals.com/sdk/build-output/vc10-20131204-5-52-50-84- vc10-dev.txt since a few weeks and I couldn't