Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

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
Download (untitled) / with headers
text/plain 745b
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?


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.