cdepillabout/pretty-simple

Compact mode destroys laziness

georgefst opened this issue · 3 comments

georgefst commented

Oh, and I realized that currently pretty-simple doesn't parse and pretty-print lazily with the compact option:

> pPrintOpt CheckColorTty defaultOutputOptionsDarkBg { outputOptionsCompact = True }  (repeat 3)

This doesn't print anything. Maybe I should make a separate issue about this...

Originally posted by @cdepillabout in #84 (comment)

georgefst commented

This is probably my fault. I'd never even thought to test it.

cdepillabout commented

Yeah, I think the real problem here is that pretty-simple doesn't have any tests checking the whole laziness thing!

georgefst commented

Yeah, I think the real problem here is that pretty-simple doesn't have any tests checking the whole laziness thing!

We have at least one! I remember from #67 (comment).

  • 👀1