Opened 12 years ago

Closed 12 years ago

#4361 closed defect (fixed)

csv driver does not recognize numeric fieldnames

Reported by: etourigny Owned by: etourigny
Priority: normal Milestone: 1.9.0
Component: OGR_SF Version: svn-trunk
Severity: normal Keywords: csv
Cc:

Description

The csv driver does not allow for numeric values to be used as field names, which is necessary but can be avoided. The solution is to search for numeric values inside quotes in the header and use them as field names. Software that generated csv files do not enclose numeric values inside double quotes, so this change should not cause problems. Fix and autotest to be committed soon.

Change History (3)

comment:1 by etourigny, 12 years ago

Resolution: → fixed
Status: new → closed

Fixed in trunk (r23445).

comment:2 by Even Rouault, 12 years ago

Resolution: fixed
Status: closed → reopened

Etienne,

You must CSLDestroy(papszTokensQuotes) at some point.

And you can drop the "(developper version)" in the html. In a few months, 1.9.0 will be the stable version ;-)

comment:3 by etourigny, 12 years ago

Resolution: → fixed
Status: reopened → closed

Thanks,

fix memleak and remove papszTokensQuotes variable (r23447)

Note: See TracTickets for help on using tickets.