Skip to content

More compact 'where's#23

Merged
georgefst merged 5 commits intomasterfrom
half-where
Sep 24, 2020
Merged

More compact 'where's#23
georgefst merged 5 commits intomasterfrom
half-where

Conversation

@georgefst
Copy link
Collaborator

Adds option to not indent 'where' bindings, instead indenting the where keyword by only half of the usual indentation step.

Closes #21.

@georgefst
Copy link
Collaborator Author

Note to self: when merging this and #26, make sure not to duplicate Main.parseBool.

{ poIndentation = fillField poIndentation,
poCommaStyle = fillField poCommaStyle
poCommaStyle = fillField poCommaStyle,
poIndentWheres = fillField poIndentWheres
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here and elsewhere, we see the diff-unfriendliness of Ormolu's style...

r' <- irTerm (sMN 0 "__foreignCall") vs env r
return (l', r')
where
splitArg tm | (_, [_, _, l, r]) <- unApply tm -- pair, two implicits
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heck yeah

@georgefst georgefst merged commit e8ada8c into master Sep 24, 2020
@brandonchinn178 brandonchinn178 deleted the half-where branch August 10, 2021 04:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Indentation of 'where'

2 participants