Bug #117158 for Encode: Encode 2.85+ breaks Encode::ISO2202JP2

Tue Aug 23 05:08:22 2016 KENTNL [...] cpan.org - Ticket created

Subject:

Encode 2.85+ breaks Encode::ISO2202JP2

See https://rt.cpan.org/Ticket/Display.html?id=117157

As of 2.85, the following block of code simply eats memory and never returns:

require Encode::ISO2022JP2;
my $text = Encode::encode( 'iso2022jp-2', chr(720) );

Tue Aug 23 05:08:52 2016 KENTNL [...] cpan.org - Reference to https://rt.cpan.org/Ticket/Display.html?id=117157 added

Tue Aug 23 05:09:47 2016 KENTNL [...] cpan.org - Reference to https://rt.cpan.org/Ticket/Display.html?id=117157 deleted

Tue Aug 23 05:09:59 2016 KENTNL [...] cpan.org - Reference to ticket #117157 added

Tue Aug 23 06:24:34 2016 KENTNL [...] cpan.org - Correspondence added

Throwing callgrind at it, the hotpoint in XS appears to start here:

https://metacpan.org/source/DANKOGAI/Encode-2.86/Encode.xs#L194

For an input of *one* character, that was called 200,000 times before I hit ^C

So if I had to guess I'd say the containing while loop never progresses enough to an exit condition.

Tue Aug 23 07:55:54 2016 hatuka [...] nezumi.nu - Status changed from 'new' to 'open'

Tue Sep 13 02:38:07 2016 ribasushi [...] leporine.io - Cc RIBASUSHI added

Wed Sep 14 05:24:08 2016 ppisar [...] redhat.com - Correspondence added

From:

ppisar [...] redhat.com

Dne Út 23.srp.2016 05:08:22, KENTNL napsal(a): Show quoted text

> See https://rt.cpan.org/Ticket/Display.html?id=117157 > > As of 2.85, the following block of code simply eats memory and never > returns: > > require Encode::ISO2022JP2; > my $text = Encode::encode( 'iso2022jp-2', chr(720) );

Encode-JISX0213 also experiences the same problem when performing its t/JISX0213.t test. -- Petr

Fri Oct 28 08:31:22 2016 pali [...] cpan.org - Cc PALI added

Fri Oct 28 08:31:52 2016 pali [...] cpan.org - Correspondence added

On Uto Aug 23 05:08:22 2016, KENTNL wrote: Show quoted text

> See https://rt.cpan.org/Ticket/Display.html?id=117157 > > As of 2.85, the following block of code simply eats memory and never > returns: > > require Encode::ISO2022JP2; > my $text = Encode::encode( 'iso2022jp-2', chr(720) );

Should be fixed in Encode 2.87.

Fri Oct 28 08:31:52 2016 pali [...] cpan.org - Fixed in 2.87 added

Tue Nov 01 09:19:29 2016 DANKOGAI [...] cpan.org - Status changed from 'open' to 'resolved'