Skip to content
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

sequence interpreted as string #9

Open
kuangdai opened this issue Apr 4, 2020 · 1 comment
Open

sequence interpreted as string #9

kuangdai opened this issue Apr 4, 2020 · 1 comment
Labels
enhancement New feature or request

Comments

@kuangdai
Copy link

kuangdai commented Apr 4, 2020

According to YMAL standard, I think the following node

foo: [1, 2, 3]

should be interpreted as a sequence, while mini-ymal interprets it as a string "[1, 2, 3]".

Is this easy to fix?

BTW: can you move #include to YAML.cpp? This is a heavy header.

@jimmiebergmann
Copy link
Owner

@kuangdai

According to YMAL standard, I think the following node

foo: [1, 2, 3]

should be interpreted as a sequence, while mini-ymal interprets it as a string "[1, 2, 3]".

Is this easy to fix?

I've not been maintaining mini-yaml for a while, due to lack of time and time spent on other projects.
Anyway, I've been planning to rewrite the parser, to use a state machine design instead of the current brute force parsing. It would probably require some work to implement json-like sequences in mini-yaml's current state.

BTW: can you move #include to YAML.cpp? This is a heavy header.

What "#include" are you referring to?

@jimmiebergmann jimmiebergmann added the enhancement New feature or request label Apr 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants