-
-
Notifications
You must be signed in to change notification settings - Fork 39
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RFC] Store inline-tag along with the tables #330
Comments
@vrom911 Excellent idea 👌 It's totally possible because we already know this information during parsing. I think, the solution to this problem will be to implement a type like: data TomlTable = TomlTable
{ tomlTableInline :: TableInline
, tomlTableTOML :: TOML
} And instead of the current type Lines 96 to 100 in 316c893
it will look like this data TOML = TOML
{ tomlPairs :: !(HashMap Key AnyValue)
, tomlTables :: !(PrefixMap TomlTable)
, tomlTableArrays :: !(HashMap Key (NonEmpty TomlTable))
} I think it's totally worth it, since it will help to produce better output! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are two ways to write tables in TOML.
But currently
tomland
doesn't keep track of such information, though it could be useful during pretty-printing later. This info is not that hard to get, as we already parsing both cases.The question is, do you think that it worth to support this enhancement? And what the downsides are?
The text was updated successfully, but these errors were encountered: