pretty-simple Compact mode destroys laziness

Compact mode destroys laziness

Open georgefst opened this issue 2 years ago • 3 comments

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 https://github.com/cdepillabout/pretty-simple/issues/84#issuecomment-1154163224

Jun 13 '22 17:06 georgefst

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

Jun 13 '22 17:06 georgefst

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

Jun 14 '22 01:06 cdepillabout

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 https://github.com/cdepillabout/pretty-simple/pull/67#issuecomment-651701234.

Jun 14 '22 08:06 georgefst