For the moment, I'm treating this as a bug in the new release. The
changes in Libmarpa would affect the NAIF as much as the SLIF. -- jeffrey
On 01/16/2014 02:11 PM, Ron Savage wrote:
The same error is manifest in a number of test files. Of course it
could well be a bug in my code.
All tests work conceptually like this:
data/15.gv -> scripts/lex.pl -> data/15.lex -> scripts/parse.pl ->
data/15.parse -> etc.
I say conceptually because the real test code (t/*.t) uses temp files
and diffs.
So, to test, just output to ./ rather than data/, and do a diff with
the pre-existing outputs in data/.
The failure is in parse.pl, which uses GraphViz2::Marpa::Parser, which
uses the NAIF. The grammar starts at line 541.
Here's how to reproduce:
curl http://savage.net.au/Perl-modules/GraphViz2-Marpa-1.12.tgz -o GM.tgz
tar xvzf GM.tgz
cd GraphViz2-Marpa-1.12/
perl -Ilib scripts/parse.pl -lex data/15.lex -parse 15.parse
diff data/15.parse 15.parse
This produces a 1 diff:
33d32
< edge_id , "->"
Likewise for 16 and 17. I did not bother checking other failures.
The problem is line 20 in data/15.gv, specifically the '->', which
appears at line 39 of data/15.lex and at line 33 of data/15.parse, but
does not appear (with the newest Marpa) at line 33 of ./15.parse.
The input data can be checked by running
perl -Ilib scripts/lex.pl -in data/15.gv -lex 15.lex
diff data/15.lex 15.lex
(Which works, i.e with no diff output)
To test all files, I do:
prove -Ilib t
--
You received this message because you are subscribed to the Google
Groups "marpa parser" group.
To unsubscribe from this group and stop receiving emails from it, send
an email to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
--
You received this message because you are subscribed to the Google Groups "marpa
parser" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.