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

LTA internals-leaky error message when doing 'for our $:: ();' in Rakudo #2775

Closed
p6rt opened this issue May 22, 2012 · 9 comments
Closed

LTA internals-leaky error message when doing 'for our $:: ();' in Rakudo #2775

p6rt opened this issue May 22, 2012 · 9 comments

Comments

@p6rt
Copy link

p6rt commented May 22, 2012

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

Searchable as RT113042$

@p6rt
Copy link
Author

p6rt commented May 22, 2012

From @masak

<masak> r​: for our $​:: ();
<p6eval> rakudo dca0fa​: OUTPUT«===SORRY!===␤ResizablePMCArray​: index
out of bounds!␤»
* masak submits LTA error diakopter++ rakudobug

@p6rt
Copy link
Author

p6rt commented May 9, 2013

From @FROGGS

still the same​:

<FROGGS> r​: for our $​:: ();
<camelia> rakudo fe7049​: OUTPUT«===SORRY!===␤ResizablePMCArray​: index
out of bounds!␤»
<FROGGS> r​: for my $​:: ();
<camelia> rakudo fe7049​: OUTPUT«===SORRY!===␤ResizablePMCArray​: index
out of bounds!␤»

@p6rt
Copy link
Author

p6rt commented May 9, 2013

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

@p6rt
Copy link
Author

p6rt commented Oct 28, 2014

From @usev6

The internal error message has changed slightly (but is still LTA)​:

$ perl6-m -e 'for our $​:: ()'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-p -e 'for our $​:: ()'
===SORRY!===
QRPA​: index out of bounds

$ perl6-j -e 'for our $​:: ()'
===SORRY!===
VMArray​: Index out of bounds

STD complains about a missing block​:

$ viv -c -e 'for our $​:: ()'
===SORRY!===
Missing block at (eval) line 1 (EOF)​:
------> for our $​:: ()⏏<EOL>
  expecting initializer
Parse failed

But it looks as if the problem ('Index out of bounds') is not related to the for loop​:

$ perl6-m -e '$​::'
===SORRY!===
MVMArray​: Index out of bounds

With other sigils the same error occurs​:

$ perl6-m -e '@​​::'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-m -e '&​::'
===SORRY!===
MVMArray​: Index out of bounds

STD gives 'syntax OK" for those three. Especially it seems to parse '$​::' the same way as '$'.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Oct 28, 2014

From @usev6

The internal error message has changed slightly (but is still LTA)​:

$ perl6-m -e 'for our $​:: ()'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-p -e 'for our $​:: ()'
===SORRY!===
QRPA​: index out of bounds

$ perl6-j -e 'for our $​:: ()'
===SORRY!===
VMArray​: Index out of bounds

STD complains about a missing block​:

$ viv -c -e 'for our $​:: ()'
===SORRY!===
Missing block at (eval) line 1 (EOF)​:
------> for our $​:: ()⏏<EOL>
  expecting initializer
Parse failed

But it looks as if the problem ('Index out of bounds') is not related to the for loop​:

$ perl6-m -e '$​::'
===SORRY!===
MVMArray​: Index out of bounds

With other sigils the same error occurs​:

$ perl6-m -e '@​​::'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-m -e '&​::'
===SORRY!===
MVMArray​: Index out of bounds

STD gives 'syntax OK" for those three. Especially it seems to parse '$​::' the same way as '$'.

@p6rt
Copy link
Author

p6rt commented Aug 29, 2015

From @coke

On Tue Oct 28 15​:16​:43 2014, bartolin@​gmx.de wrote​:

The internal error message has changed slightly (but is still LTA)​:

$ perl6-m -e 'for our $​:: ()'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-p -e 'for our $​:: ()'
===SORRY!===
QRPA​: index out of bounds

$ perl6-j -e 'for our $​:: ()'
===SORRY!===
VMArray​: Index out of bounds

STD complains about a missing block​:

$ viv -c -e 'for our $​:: ()'
===SORRY!===
Missing block at (eval) line 1 (EOF)​:
------> for our $​:: ()⏏<EOL>
expecting initializer
Parse failed

But it looks as if the problem ('Index out of bounds') is not related
to the for loop​:

$ perl6-m -e '$​::'
===SORRY!===
MVMArray​: Index out of bounds

With other sigils the same error occurs​:

$ perl6-m -e '@​​::'
===SORRY!===
MVMArray​: Index out of bounds

$ perl6-m -e '&​::'
===SORRY!===
MVMArray​: Index out of bounds

STD gives 'syntax OK" for those three. Especially it seems to parse
'$​::' the same way as '$'.

New behavior​:

15​:55 < [Coke]> r​: for our $​:: ();
15​:55 <+camelia> rakudo-{moar,jvm} 6a1879​: OUTPUT«===SORRY!=== Error while
  compiling /tmp/tmpfile␤Signatures as constraints on variables
  not yet implemented. Sorry. ␤at /tmp/tmpfile​:1␤------> for our
  $​:: ()⏏;␤»
15​:55 < GLRelia> rakudo-moar 287360​: OUTPUT«(timeout)»

15​:56 < [Coke]> m​: $​::
15​:56 <+camelia> rakudo-moar 6a1879​: OUTPUT«===SORRY!=== Error while compiling
  /tmp/qyCyFL1Z6t␤Variable '$' is not declared␤at
  /tmp/qyCyFL1Z6t​:1␤------> <BOL>⏏$​::␤»
15​:56 < GLRelia> rakudo-moar 287360​: OUTPUT«===SORRY!=== Error while compiling
  /tmp/chCtyvb9XT␤Variable '$' is not declared␤at
  /tmp/chCtyvb9XT​:1␤------> <BOL>⏏$​::␤»
--
Will "Coke" Coleda

@p6rt
Copy link
Author

p6rt commented Aug 22, 2017

From @smls

Is this still considered a BUG/LTA?

Isn't the error messages it prints now (see last comment by coke) pretty reasonable?

@p6rt
Copy link
Author

p6rt commented Sep 3, 2017

From @skids

On Tue, 22 Aug 2017 15​:35​:08 -0700, smls75@​gmail.com wrote​:

Is this still considered a BUG/LTA?

Isn't the error messages it prints now (see last comment by coke)
pretty reasonable?

I added a reminder to also handle anonymous vars in RT#​73104. The lingering issues
are just a sub-case of that problem.

Since we should not add tests for an X​::NYI on the signature constraint, and the
presence of the X​::NYI serves as ample codebase notation that the feature is
missing, I don't think keeping this ticket open serves any purpose. So I'm moving
it to 'resolved' status.

@p6rt p6rt closed this as completed Sep 3, 2017
@p6rt
Copy link
Author

p6rt commented Sep 3, 2017

@skids - 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