jnthn / grammar-debugger

Grammar::Debugger and Grammer::Tracer Perl 6 modules
36 stars 20 forks source link

Grammar::Tracer broken on new Rakudo #2

Closed moritz closed 7 years ago

moritz commented 11 years ago
$ perl6 --version
This is perl6 version 2012.11-107-g70d586f built on parrot 4.10.0 revision RELEASE_4_10_0-2-g2e66846
$ perl6 -e 'use Grammar::Tracer; grammar A { rule TOP { "foo" } }; A.parse(" foo")'
TOP
|  ws
|  orig
No such method 'MATCH' for invocant of type 'Str'
  in block  at lib/Grammar/Tracer.pm:20
  in method MATCH at src/gen/CORE.setting:10455
  in block  at lib/Grammar/Tracer.pm:20
  in regex TOP at -e:1
  in block  at lib/Grammar/Tracer.pm:16
  in method parse at src/gen/CORE.setting:10538
  in block  at -e:1
jnthn commented 11 years ago

That was fixed but latest bustage is in Grammar::Debugger:

c:\rakudo\dev\plexus>perl6 trydbg.pl
field
>
Cannot modify an immutable value
  in block  at lib/Grammar/Debugger.pm:77
  in block  at lib/Grammar/Debugger.pm:75
  in method intervene at lib/Grammar/Debugger.pm:70
  in block  at lib/Grammar/Debugger.pm:45
  in method parse at src\gen\CORE.setting:10397
  in block  at trydbg.pl:37
meisl commented 10 years ago

Fixed by #6.

hoelzro commented 9 years ago

I just tried this, and got the following:

$ perl6 -e 'use Grammar::Tracer; grammar A { rule TOP { "foo" } }; A.parse(" foo")'
TOP
* FAIL

So it's not blowing up, and it's supposed to fail because leading and trailing whitespace in a rule doesn't count for <.ws>, does it? Right @meisl and @jnthn?

meisl commented 9 years ago

Exactly. It's long been fixed and can - as well as #8 - be closed... ;-)