Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[rt.cpan.org #76373] Inversion list regression (Blead 36eb48b44 breaks Encode::JP::Mobile) #12065

Closed
p5pRT opened this issue Apr 19, 2012 · 8 comments

Comments

@p5pRT
Copy link

p5pRT commented Apr 19, 2012

Migrated from rt.perl.org#112530 (status was 'resolved')

Searchable as RT112530$

@p5pRT
Copy link
Author

p5pRT commented Apr 19, 2012

From bug-Encode-JP-Mobile@rt.cpan.org

<URL​: https://rt.cpan.org/Ticket/Display.html?id=76373 >

On Wed Apr 18 23​:31​:39 2012, SPROUT wrote​:

But I’m still getting
other test failures.

Which turn out to be a pretty serious perl regression​:

$ perl5.15.9 -e 'sub InFoo { "a\tb\n9\ta\n" } "\0" =~ /\p{InFoo}/'
panic​: attempting to append to an inversion list, but wasn't at the end of the list, final=11,
start=10, match=f at -e line 1.

caused by this commit​:

36eb48b is the first bad commit
commit 36eb48b
Author​: Karl Williamson <public@​khwilliamson.com>
Date​: Sat Nov 26 18​:24​:46 2011 -0700

  Utf8.c​: Generate and use inversion lists for binary swashes

Before that commit, it was possible to specify overlapping ranges, and perl had no trouble
with it.

@p5pRT
Copy link
Author

p5pRT commented Apr 19, 2012

From @cpansprout

On Wed Apr 18 22​:01​:02 2012, I wrote via
bug-Encode-JP-Mobile@​rt.cpan.org wrote​:

<URL​: https://rt.cpan.org/Ticket/Display.html?id=76373 >

On Wed Apr 18 23​:31​:39 2012, SPROUT wrote​:

But I’m still getting
other test failures.

Which turn out to be a pretty serious perl regression​:

$ perl5.15.9 -e 'sub InFoo { "a\tb\n9\ta\n" } "\0" =~ /\p{InFoo}/'
panic​: attempting to append to an inversion list, but wasn't at the
end of the list, final=11,
start=10, match=f at -e line 1.

caused by this commit​:

36eb48b is the first bad commit
commit 36eb48b
Author​: Karl Williamson <public@​khwilliamson.com>
Date​: Sat Nov 26 18​:24​:46 2011 -0700

Utf8\.c&#8203;: Generate and use inversion lists for binary swashes

Before that commit, it was possible to specify overlapping ranges, and
perl had no trouble
with it.

I think this should block 5.16.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Apr 19, 2012

From [Unknown Contact. See original ticket]

On Wed Apr 18 22​:01​:02 2012, I wrote via
bug-Encode-JP-Mobile@​rt.cpan.org wrote​:

<URL​: https://rt.cpan.org/Ticket/Display.html?id=76373 >

On Wed Apr 18 23​:31​:39 2012, SPROUT wrote​:

But I’m still getting
other test failures.

Which turn out to be a pretty serious perl regression​:

$ perl5.15.9 -e 'sub InFoo { "a\tb\n9\ta\n" } "\0" =~ /\p{InFoo}/'
panic​: attempting to append to an inversion list, but wasn't at the
end of the list, final=11,
start=10, match=f at -e line 1.

caused by this commit​:

36eb48b is the first bad commit
commit 36eb48b
Author​: Karl Williamson <public@​khwilliamson.com>
Date​: Sat Nov 26 18​:24​:46 2011 -0700

Utf8\.c&#8203;: Generate and use inversion lists for binary swashes

Before that commit, it was possible to specify overlapping ranges, and
perl had no trouble
with it.

I think this should block 5.16.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Apr 19, 2012

@cpansprout - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Apr 23, 2012

From @khwilliamson

Fixed by commit 9d50113
--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Apr 23, 2012

@khwilliamson - Status changed from 'open' to 'resolved'

@p5pRT p5pRT closed this as completed Apr 23, 2012
@p5pRT
Copy link
Author

p5pRT commented Apr 23, 2012

From @cpansprout

On Mon Apr 23 10​:16​:09 2012, khw wrote​:

Fixed by commit 9d50113

I can confirm that Encode​::JP​::Mobile works with it.

--

Father Chrysostomos

@p5pRT
Copy link
Author

p5pRT commented Apr 23, 2012

From [Unknown Contact. See original ticket]

On Mon Apr 23 10​:16​:09 2012, khw wrote​:

Fixed by commit 9d50113

I can confirm that Encode​::JP​::Mobile works with it.

--

Father Chrysostomos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant