Re^2: Text CSV_XS memory crash

If you already use Text::CSV_XS, there is no need to also use Text::CSV.
If you are using getline_hr () with column_names (), why not take the extra step to use bind_columns () and regain all the speed/performance you just lost by using hash references?
The comma is the default sep_char value. You should never need to pass that to the constructor.
You don't need any error_diag () call if you pass auto_diag => 1 to the constructor.

Enjoy, Have FUN! H.Merijn

Comment on Re^2: Text CSV_XS memory crash Select or Download Code

Replies are listed 'Best First'.
Re^3: Text CSV_XS memory crash by spazm (Monk) on Feb 02, 2011 at 18:40 UTC
Tux, thanks for replying to my post and reviewing my code! Yes, good point, no need to explicitly include `use Text::CSV` Because I don't want bound vars, I want a hashref. I use the hash for readability of the data and code for this test example. True, comma is the default. Leftover from modifying the original code. I'll clean that up. `auto_diag` sounds like a useful change for both scripts (mine and original poster's). changes applied.	[reply] [d/l] [select]
Re^4: Text CSV_XS memory crash by Tux (Canon) on Feb 02, 2011 at 23:17 UTC
2. You can still have a hashref: `my $csv = Text::CSV_XS->new ({ binary => 1, auto_diag => 1 }); open my $fh2, "<:encoding(utf8)", $file or die "$file: $!"; my $columns = $csv->getline ($fh2); for (@$columns) { s/^\s+//; s/\s+$//; } my %row; $csv->bind_columns (\@row{@$columns}); while ($csv->getline ($fh2)) { process_row (\%row); }` [download] Enjoy, Have FUN! H.Merijn	[reply] [d/l]
Re^5: Text CSV_XS memory crash by spazm (Monk) on Feb 05, 2011 at 21:44 UTC
thanks	[reply]