OverloadedString in source code #194

New Issue

2018-10-22T17:56:05+02:00

ruhatch commented

2018-10-22 17:56:05 +02:00

(Migrated from github.com)

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?

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?

👍 1

tfausak commented

2019-06-18 04:26:02 +02:00

(Migrated from github.com)

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.

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

Sign in to join this conversation.

No Milestone

No project

No Assignees

1 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: hexagoxel/brittany#194