search

cdepillabout / pretty-simple

pretty-printer for Haskell data types that have a Show instance
https://hackage.haskell.org/package/pretty-simple
BSD 3-Clause "New" or "Revised" License
243 stars 29 forks source link

Consider switching to 'megaparsec' from 'parsec' #29

Closed chshersh closed 6 years ago

chshersh commented 6 years ago

Difference from parsec:

cdepillabout commented 6 years ago

Hi @ChShersh, thanks for opening this issue.

I would be willing to merge a PR switching from parsec to megaparsec.

I've personally never used megaparsec in a project before, but I have had it on my radar for a while.

cdepillabout commented 6 years ago

The latest release of pretty-simple actually removed the dependency on parsec and instead just wrote the lexer by hand: #37

I'll go ahead and close this issue.

I'm not sure if the current lexer code could be made simpler by using a parsing library, but if so then someone could feel free to reopen this issue.