Lennart Spitzner

Results 140 comments of 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...