CSV driver should accept files with only one field

[Kurt Schwehr]

Proposed by Nick Harper at Google:

Propose removing this: source:trunk/gdal/ogr/ogrsf_frmts/csv/ogrcsvdatasource.cpp#L425​

    if( CSLCount(papszFields) < 2 )
	    {
	        VSIFCloseL( fp );
	        CSLDestroy( papszFields );
	        return FALSE;
	    }

Or should this become "< 1" or "!CSLCount(papszFields)"?

Example files that are proposed for writing the test case:

csv_one_column.csv

my_geometry
POINT(61.3483229561 22.6660514134)
POINT(0 0)

csv_one_column.vrt:

<OGRVRTDataSource>
  <OGRVRTLayer name="csv_one_column">
    <SrcDataSource relativeToVrt="1">csv_one_column.csv</SrcDataSource>
    <GeometryType>wkbPoint</GeometryType>
    <LayerSRS>WGS84</LayerSRS>
    <GeometryField encoding="PointFromColumns" reportSrcColumn="false"/>
  </OGRVRTLayer>
</OGRVRTDataSource>

Are we missing something? This file+vrt seems valid

[Even Rouault]

Adding a comma at the end of the header line workarounds the issue. Not sure why we don't accept one column CSV. Maybe because CSV means "Comma Separated Values" ;-)

[Even Rouault]

Still interested in this ? I can't see any drawback in the proposed change.

[Even Rouault]

This ticket has been automatically closed because Trac is no longer used for GDAL bug tracking, since the project has migrated to GitHub. If you believe this ticket is still valid, you may file it to ​https://github.com/OSGeo/gdal/issues if it is not already reported there.