-
Notifications
You must be signed in to change notification settings - Fork 19
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
ability to parse from file? #11
Comments
This is an excellent idea, and something I wasn't thinking about when writing the library. Unfortunately, that also means it will take a bit of refactoring to get it to work. Thankfully I don't think it will require too much! It's just not a quick and easy patch. I'm labelling this as an enhancement but now that TOML 0.4.0 is supported, there isn't a lot left to work on, so hopefully I can get this working soon. As always, contributions are welcome. |
I'd be willing to help where I can if you can offer some guidance on a recommended approach to refactoring |
…line report temporary removed; error reporting not tested.
Hi and thank you for this library, do you have a branch to switch git on, to rich this new feature ? |
Hey, I'm wondering how much modification would be required to enable this library to parse input from a file? Because at the moment it looks like you would need to load an entire file into memory to parse it, where as it could be more efficiently handled via reading a file with a fix sized buffer, this way memory usage would be constant (with the exception of building the TOML data structure).
For my purposes, I think reading the file into memory I can live with, but it's something I'd prefer not to have to do.
The text was updated successfully, but these errors were encountered: