Fwd: [rakudo] patch for better message when wrong syntax used for named parameter

[p6rt]

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

Searchable as RT75588$

[p6rt]

From @cognominal

Apparently sending to bitbucket is equivalent to >/dev/null

---------- Forwarded message ----------
From​: Stéphane Payrard <cognominal@​gmail.com>
Date​: Fri, Jun 4, 2010 at 1​:50 PM
Subject​: [rakudo] patch for better message when wrong syntax used for
named parameter
To​: bugs-bitbucket@​rt.perl.org

bash-3.2$ perl6

sub b($​:a) {...}
In signature parameter, placeholder variables like $​:a are illegal
you probably meant a named parameter​: '​:$a' at line 1, near ") {...}\n"

bash-3.2$ git diff src/Perl6/Actions.pm

Inline Patch

diff --git a/src/Perl6/Actions.pm b/src/Perl6/Actions.pm
index c5bbf63..6e892a4 100644
--- a/src/Perl6/Actions.pm
+++ b/src/Perl6/Actions.pm
@@ -1561,7 +1561,12 @@ method param_var($/) {
            }
        }
        elsif $twigil ne '!' && $twigil ne '.' && $twigil ne '*' {
-            $/.CURSOR.panic("Illegal to use $twigil twigil in signature");
+            my $error := "In signature parameter, '" ~ ~$/ ~ "', it

is illegal to use '" ~ $twigil ~ "' twigil"; \+            if $twigil eq '​:' \{ \+                $error := "In signature parameter, placeholder variables like " ~ ~$/ ~ " are illegal\\n" \+                           ~ "you probably meant a named parameter​: '​:" ~ $\ ~ ~$\\[0\] ~ "'"; \+            \} \+            $/\.CURSOR\.panic\($error\);         \}     \}  \}

--
cognominal stef

--
cognominal stef

[p6rt]

From @moritz

Thanks for the patch, applied as b258f5cc1633c4aa5035a3e5ba4c4dd076d50ee2.

[p6rt]

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

[p6rt]

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