Skip Menu |

This queue is for tickets about the Spreadsheet-XLSX CPAN distribution.

Report information
The Basics
Id: 54032
Status: open
Priority: 0/
Queue: Spreadsheet-XLSX

People
Owner: Nobody in particular
Requestors: ftl [...] dnv.com
Cc:
AdminCc:

Bug Information
Severity: Wishlist
Broken in: 0.12
Fixed in: (no value)



Subject: Wish: remove dependency on Text::Iconv
I presume the following code would remove the need for Text::Iconv Either include in module, or add as example in pod 'utf8::decode' requires 5.8.1 i presume. my $converter = XLSXConvert->new(); $oBook = eval{Spreadsheet::XLSX->new($file, $converter)}; { #simulate Text::IConv, using utf8::decode package XLSXConvert; sub new { my $module = shift; return bless( {}, $module ); } sub convert { my $self = shift; my ($raw) = @_; #do NOT use 'USE utf 8;', as that would be a pragma # saying this script itself is written in utf-8; #(ie. myfile.pl is encoded in utf-8) #just use the code below directly: utf8::decode($raw); return $raw; } }
It appears this has been addressed since the ticket was opened. Is this still an issue?