the Graph character class included spaces (\x20)

zevv / npeg

PEGs for Nim, another take

MIT License

330 stars 22 forks source link

the Graph character class included spaces (\x20) #4

Closed jrfondren closed 5 years ago

jrfondren commented 5 years ago

0x20 is actually ' ' (confirm with man ascii), so Graph included it erroneously, and Print included it redundantly. I've changed both classes to start at 0x21, and added a test to confirm that +Graph won't match a string beginning with a space.

zevv commented 5 years ago

Sweet, thanks!