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.

Reply via email to