Fix 197

[dasJ]

Closes #197

[github-actions]

Nixpkgs diff

[dasJ]

Diff looks like this is now in line with what we currently have in nixpkgs and nix now properly parses the string rather than failing to parse the file.

[infinisil]

Oh that's simpler than I'd have expected, neat!

[infinisil]

@dasJ Since you seem to have understood the code, can you explain why this fixes it in the commit message? Would be great to also briefly describe the problem in the commit message for future readers :)

[piegamesde]

Better put in some code comments as well, they are more visible than the git log

[dasJ]

I actually don't understand why it fixes that 🥲

[infinisil]

Ohh, I'm now seeing the problem: The formatting here is very misleading, because it makes me believe that <|> binds stronger than *>, but in reality it's the other way around: <|> is infixl 3, while *> is infixl 4.

It makes more sense when we format it this way, also added a comment:

chunk "''\\n"
<|> chunk "''\\r"
<|> chunk "''\\t"
<|> chunk "''\\'"
-- ''\ followed by any char throws away the ''\ part, leaving just the char
<|> chunk "''\\" *> (Text.singleton <$> anySingle)
<|> chunk "''$"
<|> chunk "'''"
<|> chunk "$$"
<|> try (chunk "$" <* notFollowedBy (char '{'))
<|> try (chunk "'" <* notFollowedBy (char '\''))
<|> someP (\t -> t /= '\'' && t /= '$' && t /= '\n')

[infinisil]

You nerdsniped me into fully re-implementing it 😅: #210