search

lspitzner / brittany

haskell source code formatter
GNU Affero General Public License v3.0
690 stars 72 forks source link

Braces for EmptyCases #203

Closed lspitzner closed 5 years ago

lspitzner commented 5 years ago

As mentioned over at https://github.com/haskell/haskell-ide-engine/issues/967, it may be preferable to have EmptyCases be explicit, that is:

absurd a = case a of {}
-- and the lambdacase-variant
absurd = \case {}

I think this is sensible, at least behind a config flag, and perhaps even by default.

eborden commented 5 years ago

:man_shrugging: I don't have a strong opinion on this.

lspitzner commented 5 years ago

Merged the PR as is, which applies this change, by default, without a new config flag. If anyone wants this configurable, feel free to raise an issue.