shnewto / bnf

Parse BNF grammar definitions
MIT License
258 stars 22 forks source link

Maint/issue 14 end production #19

Closed shnewto closed 7 years ago

shnewto commented 7 years ago

A couple notes on this PR that aren't captured in that last commit message.

  1. Since the ; is optional now, I removed it from the Display output, my preference is to default to not showing it.
  2. I added a nom-y macro of my own. The behavior was I interested in was processing an undefined amount of input without actually consuming any of it. nom's peek macro lets you "look ahead" but if it finds the pattern it consumes it.
coveralls commented 7 years ago

Coverage Status

Coverage increased (+0.8%) to 83.913% when pulling 82737834168dd9877e6b501a398bb70c2285373c on maint/issue-14-end-production into c30232f9c4c4b1521bd2f4fa09a1b79fd84a90c2 on 0.2.0-devel.

shnewto commented 7 years ago

closes #14