Opened 9 years ago

Closed 9 years ago

#3668 closed enhancement (fixed)

Use CPLAtofM in ArcInfo ASCII Grid importer (patch included)

Reported by: mkraemer Owned by: warmerdam
Priority: normal Milestone: 1.8.0
Component: GDAL_Raster Version: unspecified
Severity: normal Keywords:
Cc:

Description

The ArcInfo? ASCII Grid importer should use CPLAtofM to be able to handle locale dependent ASCII files. The attached patch can be used to patch frmts/aaigrid/aaigriddataset.cpp

Attachments (1)

aaigriddataset.patch (3.1 KB) - added by mkraemer 9 years ago.

Download all attachments as: .zip

Change History (5)

Changed 9 years ago by mkraemer

Attachment: aaigriddataset.patch added

comment:1 Changed 9 years ago by mkraemer

I just added a better heuristic to the patch, so the importer recognizes float values with commas automatically.

comment:2 Changed 9 years ago by Even Rouault

Are AAIGRID files with comma as decimal separator really valid ? Does ArcInfo? like them ?

comment:3 Changed 9 years ago by mkraemer

Actually, I used the ArcInfo? Toolbox command "Raster to ASCII" to generate such a file. So, the answer is yes.

By the way: the file has been created on a system with a German locale where "," is the decimal separator.

comment:4 Changed 9 years ago by Even Rouault

Component: defaultGDAL_Raster
Milestone: 1.8.0
Resolution: fixed
Status: newclosed

Fixed in trunk (r19948). Test added in r19949

Note: See TracTickets for help on using tickets.