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

Use LinkedList instead of simple Array for Node Children #1

Open
adam-rocska opened this issue Jan 8, 2024 · 0 comments
Open

Use LinkedList instead of simple Array for Node Children #1

adam-rocska opened this issue Jan 8, 2024 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@adam-rocska
Copy link
Member

For now the children of each PlistNode in the old-style parser
are stored in a simple array. This is fine for now, but to
improve type safety, it would be better to make them linked
lists instead where the stored value is actually the first item.

This way for example with dictionaries, the next item of a key
can be enforced to always be a value, the previous item can be
anything but a key, and any non-key value can only be followed
by a key. SO the compiler will fail miserably during fuckups.

@adam-rocska adam-rocska added the enhancement New feature or request label Jan 8, 2024
@adam-rocska adam-rocska self-assigned this Jan 8, 2024
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

1 participant