Data declarations

[superbobry]

Following haskell-style-guide data declarations should be indented as follows:

1. Basic case:

   data Tree a = Branch !a !(Tree a) !(Tree a)
                 | Leaf

2. Constructors with long names:

   data HttpException
       = InvalidStatusCode Int
       | MissingContentHeader

3. Records

   data Person = Person
       { firstName :: !String  -- ^ First name
       , lastName  :: !String  -- ^ Last name
       , age       :: !Int     -- ^ Age
       } deriving (Eq, Show)

The current implementations handles the first two cases correctly, the problem is with record indentation:

data Person = Person
              { firstName :: !String  -- ^ First name
              , lastName  :: !String  -- ^ Last name
              , age       :: !Int     -- ^ Age
              } deriving (Eq, Show)      

[errge]

Yes, I agree with this. For now if you put the constructor name in a newline, then it works correctly.

I've made a no in the parser todos section in the code about this.

[kuribas]

I'll look at this. May I suggest to keep issues in git, not to clutter the files? Documentation in the source is fine, and TODO list also, but debatable items or bugs can be handled more conveniently in git.

[errge]

Yes, I'm happy if we use github for discussion. I moved these bugs into the code as TODO items, because I'm almost sure that I want to implement them at some point, so it makes sense to have them here. It's much more easier to hack on a git repository locally, then click around in a browser.

For example, I haven't written anything in the code about the bug report that suggested to run a ghc process just to do indentation and communicate with it to do indentation. I don't think we will implement that soon, so no point in having it in the code.