Couple simple changes to start, including working operator with > 2 operands

[Hedrik]

some debug output added to Operator.pm and prd.pl, de-duplication of code in Parser.pm, actually print in Print.pm

[plambert]

Hey, I made some github comments on the changes.

I'm really sorry for not deleting the prl.pl file; it really should just be:

#!/bin/sh
prove

And that's it... :)

In fact, if you have installed the Prerequisites listed, you should be able
to run 'autoprove' in the directory in one terminal, and it'll re-run the
tests in t/*.t each time you update a file.

Paul

On Thu, Oct 30, 2014 at 10:49 PM, Richard A. Mead [email protected]
wrote:

some debug output added to Operator.pm and prd.pl, de-duplication of code

in Parser.pm, actually print in Print.pm

You can merge this Pull Request by running

git pull https://github.com/Hedrik/d20dsl master

Or view, comment on, or merge it at:

#1
Commit Summary

• Updated to actually display output from print statements
• more tests
• remove dup code in grammar
• add debug output

File Changes

• M DSL/Operator.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-0 (12)
• M DSL/Parser.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-1 (48)
• M DSL/Statement/Print.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-2 (3)
• M prd.pl https://github.com/plambert/d20dsl/pull/1/files#diff-3
  (19)

Patch Links:

• https://github.com/plambert/d20dsl/pull/1.patch
• https://github.com/plambert/d20dsl/pull/1.diff

Reply to this email directly or view it on GitHub
#1.

[Hedrik]

Um, OK... Where do I find these comments you made?

I try the autoprove for testing.

Not sure what you meant by munged output from multiple expressions in a
print statement... Maybe that is one of the tests? I'll check the t/
directory.

On Fri, Oct 31, 2014 at 3:11 PM, Paul M. Lambert [email protected]
wrote:

Hey, I made some github comments on the changes.

I'm really sorry for not deleting the prl.pl file; it really should just
be:

#!/bin/sh
prove

And that's it... :)

In fact, if you have installed the Prerequisites listed, you should be able
to run 'autoprove' in the directory in one terminal, and it'll re-run the
tests in t/*.t each time you update a file.

Paul

On Thu, Oct 30, 2014 at 10:49 PM, Richard A. Mead <
[email protected]>
wrote:

some debug output added to Operator.pm and prd.pl, de-duplication of
code

in Parser.pm, actually print in Print.pm

You can merge this Pull Request by running

git pull https://github.com/Hedrik/d20dsl master

Or view, comment on, or merge it at:

#1
Commit Summary

• Updated to actually display output from print statements
• more tests
• remove dup code in grammar
• add debug output

File Changes

• M DSL/Operator.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-0 (12)
• M DSL/Parser.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-1 (48)
• M DSL/Statement/Print.pm
  https://github.com/plambert/d20dsl/pull/1/files#diff-2 (3)
• M prd.pl https://github.com/plambert/d20dsl/pull/1/files#diff-3
  (19)

Patch Links:

• https://github.com/plambert/d20dsl/pull/1.patch
• https://github.com/plambert/d20dsl/pull/1.diff

Reply to this email directly or view it on GitHub
#1.

—
Reply to this email directly or view it on GitHub
#1 (comment).

[Hedrik]

Yo, these changes still look like they apply... though there is some merge work to be done perhaps?