Parser bug with record assignment

[rtfeldman]

Take a look at https://github.com/rtfeldman/dreamwriter/blob/75651371541a88a1d85504e70c7e41afd914eaae/Dreamwriter/Model.elm

Notice that emptyState at the bottom awkwardly has an extra level of indentation compared to the type alias above. It turns out this extra indentation is currently mandatory for record assignment.

If you attempt to do the "commas at the end of the line" style like this, the parser will let you do indentation "normally" in the case of type alias (as you can see earlier in the file), but if you try to use this style with record assignment (as with emptyState) and use "normal" indentation instead of awkward extra indentation, you get the following syntax error:

>> Error when compiling Dreamwriter.Model:
>> 
>> Parse error at (line 41, column 1):
>> unexpected "}"
>> expecting a newline or whitespace
Warning: Error when compiling Dreamwriter.Model:

Parse error at (line 41, column 1):
unexpected "}"
expecting a newline or whitespace

[evancz]

Please open this on the elm-compiler repo. I know about this, but have never gotten around to fixing it because it is trickier than it sounds. Currently, "whitespace" means "a bunch of spacey things that do not end with a newline" so things need to be a bit tricky to make this work given how the parser is set up.

Definitely deserves an issue though!

[evancz]

Please open this on the elm-compiler repo. I know about this, but have never gotten around to fixing it because it is trickier than it sounds. Currently, "whitespace" means "a bunch of spacey things that do not end with a newline" so things need to be a bit tricky to make this work given how the parser is set up.

Definitely deserves an issue though!

[rtfeldman]

Whoops! Created elm/compiler#842 instead.

[rtfeldman]

Whoops! Created elm/compiler#842 instead.