brittany icon indicating copy to clipboard operation
brittany copied to clipboard

OverloadedString in source code

Open ruhatch opened this issue 6 years ago • 1 comments

Right now the pattern docLit $ Text.pack "Some string" is all over the layouting functions. If we enable OverloadedStrings those can all be reduced to docLit "Some string", which would aid readability for complex layouts.

@lspitzner is there a specific reason not to use OverloadedStrings or did you just not choose to. Do you think it makes sense to enable it in a few files?

ruhatch avatar Oct 22 '18 15:10 ruhatch

I can't speak for Lennart, but I sometimes avoid OverloadedStrings because I'm not comfortable with how ByteStrings behave. That being said, it's usually worth the convenience to me.

tfausak avatar Jun 18 '19 02:06 tfausak