Skip to content
Permalink
Browse files

Use node instead of ast for placeholder AST name

Zoffix suggested this is more consistent with the Rakudo codebase
  • Loading branch information...
hoelzro committed Oct 23, 2018
1 parent 0bda7dc commit 90103f6d2426a9cdb8474a3bad4705754829eafc
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/Perl6/Actions.nqp
@@ -4337,7 +4337,7 @@ class Perl6::Actions is HLL::Actions does STDActions {
my $first-placeholder := $past.ann('placeholder_sig')[0];
my $first-placeholder-name := $first-placeholder<placeholder>;

$first-placeholder<ast>.PRECURSOR.panic("Placeholder variables (eg. $first-placeholder-name) cannot be used in a method.\nPlease specify an explicit signature, like $*METHODTYPE $method-name ($non-placeholder-names) \{ ... \}");
$first-placeholder<node>.PRECURSOR.panic("Placeholder variables (eg. $first-placeholder-name) cannot be used in a method.\nPlease specify an explicit signature, like $*METHODTYPE $method-name ($non-placeholder-names) \{ ... \}");
}

my $code := methodize_block($/, $*DECLARAND, $past, $*SIG_OBJ,
@@ -9447,7 +9447,7 @@ class Perl6::Actions is HLL::Actions does STDActions {
pos_slurpy => $pos_slurpy,
named_slurpy => $named_slurpy,
placeholder => $full_name,
ast => $/,
node => $/,
is_multi_invocant => 1,
sigil => ~$sigil);

0 comments on commit 90103f6

Please sign in to comment.
You can’t perform that action at this time.