Beefy Boxes and Bandwidth Generously Provided by pair Networks
Welcome to the Monastery
 
PerlMonks  

Re: CSV_XS issue

by Tux (Monsignor)
on Apr 07, 2013 at 10:04 UTC ( #1027351=note: print w/ replies, xml ) Need Help??


in reply to CSV_XS issue

What Corion said:

$csv = Text::CSV_XS->new ({ binary => 1, eol => "\n", auto_diag => 1 } +);

The eol attribute is only required on output. The fact that it doesn't default to $/ is because of backward compatibility. It was the way it is when I took over.


Enjoy, Have FUN! H.Merijn


Comment on Re: CSV_XS issue
Select or Download Code
Re^2: CSV_XS issue
by anaconda_wly (Scribe) on Apr 08, 2013 at 03:17 UTC
    yes, it works, thanks you both! Good to know for the history!

      Just to add a bit to clarify and keep history, as Corion asked me in /msg: the $csv object is used for both input/getline/parse and output/print/combine. Most of the questions I get are for parsing, and most of my own scripts are parsing CSV data.

      The parsing code works better when not setting eol, as it then allows more "default" values (to be mixed) and works faster (but probably not measurable).

      Setting eol to $/ by default would most likely break a lot of parser scripts. Setting eol to $/ on output only makes some sense, but also makes the documentation harder and I wonder if it would be worth breaking backward compatibility.

      What does make sense in breaking backward compatibility is to make binary default to a true value, as I can't think of a modern situation where it would break a parser.


      Enjoy, Have FUN! H.Merijn
      cc

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://1027351]
help
Chatterbox?
and the web crawler heard nothing...

How do I use this? | Other CB clients
Other Users?
Others scrutinizing the Monastery: (12)
As of 2014-12-27 16:43 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?

    Is guessing a good strategy for surviving in the IT business?





    Results (177 votes), past polls