Lennart Spitzner
Lennart Spitzner
Does this apply only to function applications? The brittany layouting construct connected to this would be "par-spacing", i.e. the `docSetParSpacing` and `docForceParSpacing` bits of the layouting DSL, where `HsApp` is...
phrased differently: the node to the right of "=" is currently allowed to have "par-spacing" and several layouters currently set the required "bit", and `HsApp` is only one of them.
Sorry, I never followed up on this either. It still is on my to-do list to find those testcases you asked for, when I find time.
@chreekat Finally found some time (sorry). And after trying to find the difference line 183 makes, I have to admit that it probably is indeed superfluous, and this comes down...
You probably have not noticed the branch "datadecl" yet; it already contains some (unfinished) initial work in this direction. Although I notice I have a commit that is not even...
Ok, pushed. It is good news that you want to contribute and I should be able to give you some more specific hints at how to approach the task.
Firstly, my workflow in general when enhancing brittany is: 1) Think of some short example code that contains the not-yet-layouted syntax; 2) pass that to `brittany --dump-ast-full` 3) Find out...
Some other random thoughts: - regarding reading the GHC API: In this context, all "PostRn" stuff can be ignored. - master now works on both ghc-8.0 and 8.2, but of...
Oh, and to answer the initial question: It should not be necessary to touch any code other than `Language.Haskell.Brittany.Internal.Layouters.*` when adding support for other parts of ghc syntax. The structure...
Thinking some more, perhaps it is not even good that the initial work is already done - to get familiar with the interface it might be better to start by...