Skip Menu |

This queue is for tickets about the DBD-CSV CPAN distribution.

Report information
The Basics
Id: 33767
Status: resolved
Priority: 0/
Queue: DBD-CSV

People
Owner: Nobody in particular
Requestors: IKEGAMI [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.22
Fixed in: 0.24_01



DBD::CSV accesses the guts of the helper CSV class instead of using the documented interface. This causes problems when using a class other than Text::CSV. local $/ = $csv->{'eol'}; should be local $/ = $csv->eol();
On Mon Mar 03 00:49:05 2008, ikegami wrote: Show quoted text
> DBD::CSV accesses the guts of the helper CSV class instead of using
the Show quoted text
> documented interface. This causes problems when using a class other
than Show quoted text
> Text::CSV. > > local $/ = $csv->{'eol'}; > should be > local $/ = $csv->eol();
Correct, but worse, why does DBD::CSV use eol at all for reading? the eol attribute is (very) useful for writing CSV, but it takes away all the flexibility in parsing CSV, which will go file for \r, \n *AND* \r \n by default. When I tried today, the file would not open without having me to explicitely pass csv_eol=\n