Re: [gdal-dev] S57 field definition incorrect

2009-01-15 Thread ogr user
In case anyone is interested, here is a patch that adds appropriate handling of IntegerList field types to S57 parser. For backwards compatibility it is only activated if specific option is passed to the reader (actually to class registrar). This is for 1.5.3. Frank Warmerdam wrote: > > Dear OGR U

Re: [gdal-dev] S57 field definition incorrect

2009-01-14 Thread Frank Warmerdam
ogr user wrote: I see that now in code. I added my own wrapper to parse the list, but there are two issues remaining: 1) efficiency - if a field must be read multiple times, string will have to be parsed multiple times, whereas if it were treated as integer list from the start, string parsing wou

Re: [gdal-dev] S57 field definition incorrect

2009-01-14 Thread ogr user
I see that now in code. I added my own wrapper to parse the list, but there are two issues remaining: 1) efficiency - if a field must be read multiple times, string will have to be parsed multiple times, whereas if it were treated as integer list from the start, string parsing would be done once. T

Re: [gdal-dev] S57 field definition incorrect

2009-01-13 Thread Frank Warmerdam
ogr user wrote: I am using OGR to load S57 format files and it appears that at least for some fields the field type is incorrectly returned by OGR. Fields of type "integer list" (denoted L in s57attributes.csv, enumeration list) are identified as string type by OGRField::GetType(). The problem w

[gdal-dev] S57 field definition incorrect

2009-01-13 Thread ogr user
I am using OGR to load S57 format files and it appears that at least for some fields the field type is incorrectly returned by OGR. Fields of type "integer list" (denoted L in s57attributes.csv, enumeration list) are identified as string type by OGRField::GetType(). The problem with that is that O