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

Use of undefined routines not detected at compile time #2043

Closed
p6rt opened this issue Aug 9, 2010 · 6 comments
Closed

Use of undefined routines not detected at compile time #2043

p6rt opened this issue Aug 9, 2010 · 6 comments

Comments

@p6rt
Copy link

p6rt commented Aug 9, 2010

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

Searchable as RT77112$

@p6rt
Copy link
Author

p6rt commented Aug 9, 2010

From @bbkr

I cannot find if this was already reported​:

when you use sub in {} part in substitution lack of this sub is detected in
parse time in STD​:

[16​:25] <bbkr> std​: $_ = ""; s/1/{foo()}/; # curious if it will find undefined
sub in parsetime
[16​:25] <p6eval> std 31912​: OUTPUT«�[31m===�[0mSORRY!�[31m===�[0mâ�¤Undeclared
routine​:â�¤ 'foo' used at line 1â�¤Check failedâ�¤FAILED 00​:02 116mâ�¤Â»

but it parses on Rakudo​:

[16​:25] <bbkr> rakudo​: $_ = ""; s/1/{foo()}/; # curious if it will find
undefined sub in parsetime
[16​:25] <p6eval> rakudo c1e19a​: ( no output )

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

From @moritz

This is a general problem in rakudo, not just in the specific case
mentioned here.

@p6rt
Copy link
Author

p6rt commented Oct 8, 2011

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

@p6rt
Copy link
Author

p6rt commented Nov 15, 2011

From @tadzik

On Sat Oct 08 11​:05​:14 2011, moritz wrote​:

This is a general problem in rakudo, not just in the specific case
mentioned here.

Works now​:

$_ = ""; s/1/{foo()}/;
CHECK FAILED​:
Undefined routine '&foo' called (line 1)

Tests needed, I suppose

@p6rt
Copy link
Author

p6rt commented Jan 14, 2012

From @moritz

Now tested in t/spec/S04-declarations/my.t

@p6rt
Copy link
Author

p6rt commented Jan 14, 2012

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

@p6rt p6rt closed this as completed Jan 14, 2012
@p6rt p6rt added the testneeded label Jan 5, 2020
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