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

duplicate rule/token names break grammar parsing #2786

Closed
p6rt opened this issue Jun 8, 2012 · 5 comments
Closed

duplicate rule/token names break grammar parsing #2786

p6rt opened this issue Jun 8, 2012 · 5 comments

Comments

@p6rt
Copy link

p6rt commented Jun 8, 2012

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

Searchable as RT113552$

@p6rt
Copy link
Author

p6rt commented Jun 8, 2012

From @diakopter

00​:28 < diakopter> r​: grammar A { token a { }; token a { } }
00​:28 <+p6eval> rakudo 6ab416​: OUTPUT«===SORRY!===␤Method 'match' not found for
  invocant of class 'String'␤at /tmp/TaBvwFfZ6b​:1␤»

expected​: a real error.

@p6rt
Copy link
Author

p6rt commented Jun 11, 2012

From @jnthn

On Thu Jun 07 22​:49​:12 2012, diakopter.gmail.com wrote​:

00​:28 < diakopter> r​: grammar A { token a { }; token a { } }
00​:28 <+p6eval> rakudo 6ab416​: OUTPUT«===SORRY!===␤Method 'match' not
found for
invocant of class 'String'␤at /tmp/TaBvwFfZ6b​:1␤»

expected​: a real error.

Now it gives the usual duplicate method error, as it was meant to in the
first place.

===SORRY!===
Package 'A' already has a method 'a' (did you mean to declare a
multi-method?)

Thanks,

Jonathan

@p6rt
Copy link
Author

p6rt commented Jun 11, 2012

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

@p6rt
Copy link
Author

p6rt commented Jan 15, 2014

From @coke

On Mon Jun 11 12​:47​:49 2012, jnthn@​jnthn.net wrote​:

On Thu Jun 07 22​:49​:12 2012, diakopter.gmail.com wrote​:

00​:28 < diakopter> r​: grammar A { token a { }; token a { } }
00​:28 <+p6eval> rakudo 6ab416​: OUTPUT«===SORRY!===␤Method 'match' not
found for
invocant of class 'String'␤at /tmp/TaBvwFfZ6b​:1␤»

expected​: a real error.

Now it gives the usual duplicate method error, as it was meant to in the
first place.

===SORRY!===
Package 'A' already has a method 'a' (did you mean to declare a
multi-method?)

Thanks,

Jonathan

Error has changed slightly to complain about a Regex instead of a method.

Test added to S05-grammar/methods.t

--
Will "Coke" Coleda

@p6rt p6rt closed this as completed Jan 15, 2014
@p6rt
Copy link
Author

p6rt commented Jan 15, 2014

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

@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