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

Can't take index 100_000 of constant list in Rakudo Moar #3487

Closed
p6rt opened this issue Aug 24, 2014 · 7 comments
Closed

Can't take index 100_000 of constant list in Rakudo Moar #3487

p6rt opened this issue Aug 24, 2014 · 7 comments

Comments

@p6rt
Copy link

p6rt commented Aug 24, 2014

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

Searchable as RT122604$

@p6rt
Copy link
Author

p6rt commented Aug 24, 2014

From @masak

<jimmyz> r​: constant fib = 0, 1, *+* ... *; say fib[100000]
<camelia> rakudo-{parrot,jvm} c793d8​: OUTPUT«(timeout)»
<camelia> ..rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<jimmyz> fib[99999] works
<jimmyz> masak​: ^
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[99999]
<camelia> rakudo-moar c793d8​:
OUTPUT«1605285768272981969703501699166323590285867163980464844832239016803082277106148858007382799983341986487069902980550885975477322351434706230637004409049784072899949196507083084156794116251699801486721736223390722427570186888655751529235613410338408870193727…»
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> yeah, that seems like an arbitrary limit...
<masak> m​: constant fib = 0, 1 ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> m​: constant fib = 0 xx 100_001; say fib[99_999]
<camelia> rakudo-moar c793d8​: OUTPUT«0␤»
<masak> m​: constant fib = 0 xx 100_002; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> ok, so it doesn't necessarily involve infix​:<...>

@p6rt
Copy link
Author

p6rt commented Aug 26, 2014

From @coke

On Sat Aug 23 21​:09​:03 2014, masak wrote​:

<jimmyz> r​: constant fib = 0, 1, *+* ... *; say fib[100000]
<camelia> rakudo-{parrot,jvm} c793d8​: OUTPUT«(timeout)»
<camelia> ..rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<jimmyz> fib[99999] works
<jimmyz> masak​: ^
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[99999]
<camelia> rakudo-moar c793d8​:
OUTPUT«1605285768272981969703501699166323590285867163980464844832239016803082277106148858007382799983341986487069902980550885975477322351434706230637004409049784072899949196507083084156794116251699801486721736223390722427570186888655751529235613410338408870193727…»
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> yeah, that seems like an arbitrary limit...
<masak> m​: constant fib = 0, 1 ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> m​: constant fib = 0 xx 100_001; say fib[99_999]
<camelia> rakudo-moar c793d8​: OUTPUT«0␤»
<masak> m​: constant fib = 0 xx 100_002; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> ok, so it doesn't necessarily involve infix​:<...>

20​:07 <+dalek> rakudo/nom​: ce0c258 | TimToady++ | src/core/ListIter.pm​:
20​:07 <+dalek> rakudo/nom​: Reify lists with correct count.
20​:07 <+dalek> rakudo/nom​:
20​:07 <+dalek> rakudo/nom​: The ListIter reify called p6arrfindtypes with $max
  (which is 100000), but should
20​:07 <+dalek> rakudo/nom​: be using $count, how many we really want.
20​:07 <+dalek> rakudo/nom​: Fixes RT #​122604.
20​:07 < synopsebot> Link​:
  https://rt-archive.perl.org/perl6//Public/Bug/Display.html?id=122604
20​:07 <+dalek> rakudo/nom​: review​:
  rakudo/rakudo@ce0c258640

Closable with tests.

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Aug 26, 2014

The RT System itself - Status changed from 'new' to 'open'

@p6rt
Copy link
Author

p6rt commented Aug 27, 2014

From @coke

On Tue Aug 26 07​:49​:26 2014, coke wrote​:

On Sat Aug 23 21​:09​:03 2014, masak wrote​:

<jimmyz> r​: constant fib = 0, 1, *+* ... *; say fib[100000]
<camelia> rakudo-{parrot,jvm} c793d8​: OUTPUT«(timeout)»
<camelia> ..rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<jimmyz> fib[99999] works
<jimmyz> masak​: ^
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[99999]
<camelia> rakudo-moar c793d8​:
OUTPUT«1605285768272981969703501699166323590285867163980464844832239016803082277106148858007382799983341986487069902980550885975477322351434706230637004409049784072899949196507083084156794116251699801486721736223390722427570186888655751529235613410338408870193727…»
<masak> m​: constant fib = 0, 1, *+* ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> yeah, that seems like an arbitrary limit...
<masak> m​: constant fib = 0, 1 ... *; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> m​: constant fib = 0 xx 100_001; say fib[99_999]
<camelia> rakudo-moar c793d8​: OUTPUT«0␤»
<masak> m​: constant fib = 0 xx 100_002; say fib[100_000]
<camelia> rakudo-moar c793d8​: OUTPUT«P6opaque​: no such attribute
'$!storage' [...]
<masak> ok, so it doesn't necessarily involve infix​:<...>

20​:07 <+dalek> rakudo/nom​: ce0c258 | TimToady++ |
src/core/ListIter.pm​:
20​:07 <+dalek> rakudo/nom​: Reify lists with correct count.
20​:07 <+dalek> rakudo/nom​:
20​:07 <+dalek> rakudo/nom​: The ListIter reify called p6arrfindtypes
with $max
(which is 100000), but should
20​:07 <+dalek> rakudo/nom​: be using $count, how many we really want.
20​:07 <+dalek> rakudo/nom​: Fixes RT #​122604.
20​:07 < synopsebot> Link​:
https://rt-archive.perl.org/perl6//Public/Bug/Display.html?id=122604
20​:07 <+dalek> rakudo/nom​: review​:
rakudo/rakudo@ce0c258640

Closable with tests.

Test added in S04-declarations/constant.t

--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Aug 27, 2014

@coke - Status changed from 'open' to 'deleted'

@p6rt
Copy link
Author

p6rt commented Aug 27, 2014

@coke - Status changed from 'deleted' to 'open'

@p6rt
Copy link
Author

p6rt commented Aug 27, 2014

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

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

No branches or pull requests

1 participant