Evan Rutledge Borden

Results 62 comments of Evan Rutledge Borden

@lpil the issue tracking it :point_right: https://github.com/lspitzner/brittany/issues/145

There are three areas where space can exist, before the first bracket, after the first bracket and before the second bracket. The reasonable options are: ```hs Record{..} Record {..} Record...

I'm not against the "big change" as it binds the record identifier tightly with the opening bracket. For the spacious bracket you'd have: ```hs match Record {} = 1 match...

@lspitzner one way to avoid inconsistency on your previous layout is to have spacious wildcards, but tight identifier/bracket binding. ```hs match Record{} = 1 match Record{ .. } = 2...

This is the big win for me. The fact that record update/construction binds tighter than application makes removing the spurious space (between the identifier and the opening bracket) a positive...

@lspitzner given your :+1:, it looks like you endorse this layout: https://github.com/lspitzner/brittany/issues/251#issuecomment-527611371 ![image](https://user-images.githubusercontent.com/545655/64477177-e1162780-d15d-11e9-814f-ae64c5019e7d.png) @simonchatts has proposed opening a PR, should he move forward with this layout?

> As we are still pre 1.0, I say we accept this breaking change, even without making it configurable. Should people dislike this change, we can always come back and...

Variations ```yaml conf_layout: lconfig_experimental: true ``` or ```yaml conf_layout: lconfig_unstable: true ```

:point_up: unnassigned since `assignees` will be used for active work.

@mrkgnao `brittany` does this in a number of places, but you'll generally be looking in `src/Language/Haskell/Brittany/Internal/Layouters/.`