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
Null PMC access when calling Grammar.WALK in Rakudo #1038
Comments
From @masak<jnthn> rakudo: my ($meth) = Grammar.WALK(:name<parse>); |
From @kylehaThis is an automatically generated mail to inform you that tests are now available in t/spec/S12-introspection/walk.t commit cc81d63c2c6fd91dd85c16c45c0862fd2a26f040 [t/spec] Add a test for RT#66330 to walk.t. Also tweak a couple of tests that won't quite work; things passed into :omit and :include need to be a little more careful since they'll be invoked with Mu at some point, which in this case didn't end too well. Inline Patchdiff --git a/t/spec/S12-introspection/walk.t b/t/spec/S12-introspection/walk.t
index 8737870..c83c50d 100644
--- a/t/spec/S12-introspection/walk.t
+++ b/t/spec/S12-introspection/walk.t
@@ -2,7 +2,7 @@ use v6;
use Test;
-plan 10;
+plan 11;
=begin pod
@@ -91,17 +91,21 @@ sub cand_order(@cands, $instance) {
# :include
{
my $x = E.new;
- my @cands = $x.WALK(:name<m>, :include(regex { <[CDE]> }));
- #?rakudo skip ':include fails'
+ my @cands = $x.WALK(:name<m>, :include({ ~$^c ~~ regex { <[CDE]> } }));
is cand_order(@cands, $x), 'ECD', ':include works';
}
# :include and :omit
{
my $x = E.new;
- my @cands = $x.WALK(:name<m>, :include(regex { <[CDE]> }), :omit({ .^can('n') }));
- #?rakudo skip ':include/:omit together fail'
+ my @cands = $x.WALK(:name<m>, :include({ ~$^c ~~ regex { <[CDE]> } }), :omit({ .^can('n') }));
is cand_order(@cands, $x), 'D', ':include and :omit together work';
}
+# Grammar.WALK had issues once
+{
+ my ($meth) = Grammar.WALK(:name<parse>);
+ is $meth.name, 'parse', 'Grammar.WALK works';
+}
+
# vim: ft=perl6 |
The RT System itself - Status changed from 'new' to 'open' |
From @jnthnOn Fri Jun 05 03:05:41 2009, masak wrote:
Works now:
And added a test to walk.t, so resolving. Thanks, Jonathan |
@jnthn - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#66330 (status was 'resolved')
Searchable as RT66330$
The text was updated successfully, but these errors were encountered: